Skip to content

Conversation

@cvinh
Copy link

@cvinh cvinh commented Jan 17, 2026

Use case :
We have 2 databases dbname and dbname-test
In case we setup the dbfilter to dbname, we have 2 databases that match.
So we are donig a strict match now (re.fullmatch since python 3.4)

Copy link
Member

@hbrunn hbrunn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is supposed to do exactly what https://github.com/OCA/OCB/blob/18.0/odoo/http.py#L380 does but for headers, so I disagree with this change.

Your case should be handled with appending a $ sign, just as you'd do it if the dbfilter was in the config file. As this seems to be unclear, I suggest to change your PR to an amendment of the readme clarifying this.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants