Skip to content

fix: improve socket validation#9860

Open
Rello wants to merge 5 commits intomasterfrom
Rello-patch-1
Open

fix: improve socket validation#9860
Rello wants to merge 5 commits intomasterfrom
Rello-patch-1

Conversation

@Rello
Copy link
Copy Markdown
Collaborator

@Rello Rello commented Apr 17, 2026

fix: improve socket validation

@Rello Rello marked this pull request as draft April 17, 2026 10:39
@Rello Rello added this to the 33.0.3 milestone Apr 17, 2026
@Rello Rello marked this pull request as ready for review April 20, 2026 12:53
Rello and others added 5 commits April 23, 2026 10:36
@Rello @mgallien
fix: improve socket validation
779c29b 
Signed-off-by: Rello <Rello@users.noreply.github.com>
@Rello @mgallien
Fix indentation in socketapi.cpp
6b25679 
Signed-off-by: Rello <Rello@users.noreply.github.com>
@mgallien
fix(share): ask for confirmation before leaving a share
121a9c5 
there is no way to undo leaving a share. Better ask users for
confirmation

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@Rello @mgallien
Update socketapi.cpp
6e95578 
Signed-off-by: Rello <Rello@users.noreply.github.com>
@mgallien
fix(leaveShare): do not interact with user in automated tests
49c4bb4 
automated tests of leaveShare are run without graphical user interaction
possibnilities

ensure the tests will not try to create QWidget items preventing a crash

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2026

Artifact containing the AppImage: nextcloud-appimage-pr-9860.zip

Digest: sha256:dfb3b931ce9f0b7a906aca615bc89898b1c0e9a56de39a795d6cecaa32e89ef1

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 23, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
25.0% Coverage on New Code (required ≥ 80%)
76 New Code Smells (required ≤ 0)
D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@mgallien
Copy link
Copy Markdown
Collaborator

mgallien commented Apr 23, 2026

for now, I had an issue with this PR

2026-04-23 12:10:38:866 [ warning nextcloud.gui.socketapi C:\Nextcloud\desktop\src\gui\socketapi\socketapi.cpp:239 ]:	Could not send all data on socket for  "REGISTER_PATH:C:\\Users\\Matthieu Gallien\\Nextcloud7\n"
2026-04-23 12:10:42:178 [ critical default C:\Nextcloud\desktop\src\gui\socketapi\socketapi.cpp:331 ]:	ASSERT: "socket->readAll().isEmpty()" in file C:\Nextcloud\desktop\src\gui\socketapi\socketapi.cpp, line 331
2026-04-23 12:10:42:178 [ warning nextcloud.gui.socketapi C:\Nextcloud\desktop\src\gui\socketapi\socketapi.cpp:239 ]:	Could not send all data on socket for  "REGISTER_PATH:C:\\Users\\Matthieu Gallien\\Nextcloud7\n"

@mgallien
Copy link
Copy Markdown
Collaborator

mgallien commented Apr 23, 2026

will need investigation

@mgallien mgallien modified the milestones: 33.0.3, 33.0.4 Apr 27, 2026
@nilsding nilsding modified the milestones: 33.0.4, 33.0.5 May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Aiiaiiio Aiiaiiio Awaiting requested review from Aiiaiiio

@camilasan camilasan Awaiting requested review from camilasan camilasan is a code owner

@i2h3 i2h3 Awaiting requested review from i2h3 i2h3 is a code owner

@mgallien mgallien Awaiting requested review from mgallien mgallien is a code owner

@nilsding nilsding Awaiting requested review from nilsding nilsding is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@mgallien mgallien

Labels

None yet

Projects

None yet

Milestone

33.0.5

Development

Successfully merging this pull request may close these issues.

3 participants

@Rello @mgallien @nilsding