Skip to content

[17.0][FIX] currency_rate_update_xe: Obtain the values without locks + tests - #242

Merged
OCA-git-bot merged 2 commits into
OCA:17.0from
Tecnativa:17.0-fix-currency_rate_update_xe-TT65060
Oct 19, 2026
Merged

OCA-git-bot merged 2 commits into
OCA:17.0from
Tecnativa:17.0-fix-currency_rate_update_xe-TT65060

Conversation

@victoralmau

@victoralmau victoralmau commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Backport from 18.0: #241

Superseed #234

Obtain the values without locks + tests

Please @pedrobaeza and @sergio-teruel can you review it?

@Tecnativa TT65060

mohamedhagag and others added 2 commits October 19, 2026 13:52
xe.com started blocking requests without a User-Agent header around
May 2025, returning HTTP 403 which silently produced empty rate results.
Add a browser User-Agent, call raise_for_status() so errors surface
as a proper UserError, and switch the base URL to HTTPS.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@OCA-git-bot OCA-git-bot added mod:currency_rate_update_xe Module currency_rate_update_xe series:17.0 labels Oct 19, 2026
@pedrobaeza pedrobaeza added this to the 17.0 milestone Oct 19, 2026
@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-242-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 21feb75. Thanks a lot for contributing to OCA. ❤️

@OCA-git-bot
OCA-git-bot merged commit d4f3f3c into OCA:17.0 Oct 19, 2026
5 of 7 checks passed
@pedrobaeza
pedrobaeza deleted the 17.0-fix-currency_rate_update_xe-TT65060 branch October 19, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants