Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update User-Agent #20801

Merged
merged 3 commits into from
May 10, 2024
Merged

Update User-Agent #20801

merged 3 commits into from
May 10, 2024

Conversation

cayenne17
Copy link
Contributor

@cayenne17 cayenne17 commented May 4, 2024

Proposal fix problem download RSS feed fil : #13446

I changed the user agent

I'm not a developer

cayenne17 added 2 commits May 4, 2024 11:51
Proposal fix problem download RSS feed flihttps://github.com/qbittorrent/issues/13446

I'm not a developer
Proposal fix problem download RSS feed flihttps://github.com/qbittorrent/issues/13446

I'm not a developer
@xavier2k6 xavier2k6 requested a review from a team May 4, 2024 10:19
@cayenne17
Copy link
Contributor Author

@xavier2k6 : So we're using Firefox version 115 instead?
The code I've modified concerns HTTP/HTTPS requests and not restrictions?

@glassez
Copy link
Member

glassez commented May 4, 2024

I'm not sure if we should use latest "Firefox" agent as we still support Latest active ESR release for Firefox.

I don't think it's related. It's just what user-agent qBittorrent appears to be when making HTTP requests.

@xavier2k6
Copy link
Member

So we're using Firefox version 115 instead?

That's what my thoughts were initially but you can disregard it now.

@xavier2k6
Copy link
Member

It's just what user-agent qBittorrent appears to be when making HTTP requests.

I was just thinking along the lines of that we could've used the ESR agent still supported ie 115 or we could've made it random from a list of user-agents from the supported browsers but "Yes, I was over-thinking/over-complicating a simple solution"

@glassez
Copy link
Member

glassez commented May 4, 2024

It would be nice to use something that would not require updating this hardcoded version too often.

@cayenne17
Copy link
Contributor Author

@glassez https://product-details.mozilla.org/1.0/firefox_versions.json
before the build, retrieve the latest ESR version and modify the code dynamically?

@xavier2k6
Copy link
Member

Bump to 1.44


It would be nice to use something that would not require updating this hardcoded version too often.

(~5yrs ago) was last time agent string was updated, in PR #11049

change version number
@cayenne17
Copy link
Contributor Author

Bump to 1.44

It would be nice to use something that would not require updating this hardcoded version too often.

(~5yrs ago) was last time agent string was updated, in PR #11049

I've just made a commit to change the version number

@glassez glassez changed the title Proposal fix problem download RSS feed fil Update User-Agent May 5, 2024
@glassez glassez added Core Search engine Issues related to the search engine/search plugins functionality labels May 5, 2024
@glassez glassez modified the milestones: 4.6.5, 5.0 May 5, 2024
@glassez glassez requested a review from a team May 5, 2024 06:18
@glassez glassez merged commit 21a4ab6 into qbittorrent:master May 10, 2024
10 of 14 checks passed
@glassez
Copy link
Member

glassez commented May 10, 2024

@cayenne17
Thank you!

@cayenne17
Copy link
Contributor Author

@glassez @xavier2k6 : when is the patch due to be deployed on 4.6.x?

It hasn't been deployed on version 4.6.5, which was released after this PR was merged.

@xavier2k6
Copy link
Member

@glassez Please add this to #21035 Ref.: #20864 (comment) & it will close ticket #13446

glassez pushed a commit to glassez/qBittorrent that referenced this pull request Aug 15, 2024
@glassez
Copy link
Member

glassez commented Aug 15, 2024

@glassez Please add this to #21035 Ref.: #20864 (comment) & it will close ticket #13446

Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core Search engine Issues related to the search engine/search plugins functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants