Skip to content

use application name as user agent - #6401

Merged
mahibi merged 2 commits into
masterfrom
useApplicationNameAsUserAgent
Jul 3, 2026
Merged

use application name as user agent#6401
mahibi merged 2 commits into
masterfrom
useApplicationNameAsUserAgent

Conversation

@mahibi

@mahibi mahibi commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

How to test

see http requests in logcat.
For all requests there should be
User-Agent: Mozilla/5.0 (Android) Nextcloud-Talk v24.1.0 Alpha 15
Only for the login request there should be
User-Agent: Mozilla/5.0 (Android) Nextcloud Talk v24.1.0 Alpha 15 (can also be seen at the login website)

so it's "Nextcloud-Talk" vs "Nextcloud Talk".

Seems meaningless when it's the native Nextcloud Talk client, but it's important for branded clients.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@mahibi mahibi added this to the 24.1.0 milestone Jul 2, 2026
@mahibi mahibi self-assigned this Jul 2, 2026
@mahibi mahibi added the 3. to review Waiting for reviews label Jul 2, 2026
@mahibi

mahibi commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator Author

/backport to stable-24.0

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/28600595098/artifacts/8043539273
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@mahibi
mahibi marked this pull request as draft July 2, 2026 16:58
@mahibi mahibi removed the 3. to review Waiting for reviews label Jul 2, 2026
@mahibi
mahibi force-pushed the useApplicationNameAsUserAgent branch from caa0641 to 9c90d80 Compare July 2, 2026 17:16
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/28608526275/artifacts/8046616136
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

..important for branding

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
@mahibi
mahibi force-pushed the useApplicationNameAsUserAgent branch from 9c90d80 to a54efda Compare July 3, 2026 08:36
@mahibi
mahibi marked this pull request as ready for review July 3, 2026 08:36
@mahibi mahibi added the 3. to review Waiting for reviews label Jul 3, 2026
"testing anonymouslyPostLoginRequest correct path"

❯ ERROR: NetworkLoginDataSource: Error caught at anonymouslyPostLoginRequest: java.lang.NullPointerException
junit.framework.AssertionFailedError
at junit.framework.Assert.fail(Assert.java:55)
at junit.framework.Assert.assertTrue(Assert.java:22)
at junit.framework.Assert.assertNotNull(Assert.java:256)
at junit.framework.Assert.assertNotNull(Assert.java:248)
at junit.framework.TestCase.assertNotNull(TestCase.java:391)
at com.nextcloud.talk.login.data.network.NetworkLoginDataSourceTest.testing anonymouslyPostLoginRequest correct path(NetworkLoginDataSourceTest.kt:51)

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
@mahibi
mahibi merged commit 567b6f1 into master Jul 3, 2026
13 of 19 checks passed
@mahibi
mahibi deleted the useApplicationNameAsUserAgent branch July 3, 2026 11:29
@backportbot

backportbot Bot commented Jul 3, 2026

Copy link
Copy Markdown

The backport to stable-24.0 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable-24.0
git pull origin stable-24.0

# Create the new backport branch
git checkout -b backport/6401/stable-24.0

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick a54efdab e72de223

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/6401/stable-24.0

Error: Failed to clone repository: Failed to create working tree: Preparing worktree (new branch 'backport/6401/stable-24.0')
fatal: not a valid object name: 'origin/stable-24.0'


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@mahibi

mahibi commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator Author

/backport to stable-24.0.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants