Skip to content

New Release v2.8.0 - #minor#71

Merged
ltshb merged 10 commits intomasterfrom
develop
Mar 30, 2026
Merged

New Release v2.8.0 - #minor#71
ltshb merged 10 commits intomasterfrom
develop

Conversation

@hbollon
Copy link
Copy Markdown
Member

@hbollon hbollon commented Feb 4, 2026

Add otel instrumentation and monkey patching

@github-actions github-actions bot changed the title New release New Release v2.8.0 - #minor Feb 4, 2026
@hbollon hbollon requested a review from ltshb February 4, 2026 09:25
ltshb and others added 5 commits February 9, 2026 07:36
Currently, the regex for preventing qrcoding URLs other than map.geo.admin
can be bypassed, e.g. by map.geo.admin.ch.attacker.com, since the regex is too weak.
Hence the regex was anchored to make sure, that the URL to be qrcoded is exactly
map.geo.admin.ch, rather than checking, that the string is somewhere in the URL.
Using fullmatch() instead of match() only makes the service safer.
This way, even if the regex is not anchored, map.geo.admin.ch.evil.com
bypass trick will still be detected by the fullmatch and blocked.
Otherwise the regex needs to be anchored.
…allback

PB-2205: using re.fullmatch() instead if re.match()
@hbollon
Copy link
Copy Markdown
Member Author

hbollon commented Mar 27, 2026

This PR staled a little bit due to discussions and changes around OTEL config. It should be okay now and it will be released on monday.

@ltshb
Copy link
Copy Markdown
Contributor

ltshb commented Mar 30, 2026

@hbollon I'm already merging this to have it on INT already

@ltshb ltshb merged commit 51e397d into master Mar 30, 2026
4 of 5 checks passed
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.

3 participants