post://
posts://
gets://
etc notification paths are double-encoded
#2868
Labels
bug
Something isn't working
Notifications systems
Development of notifications of changes
triage
upstream-bug
issue with library - not directly with changedetection
Describe the bug
If you use a notification path with a space or escape sequence, such as:
posts://example.com/my endpoint
posts://example.com/my%20endpoint
Then the notification will be sent to:
posts://example.com/my%2520endpoint
This will likely fail with a 404 or other error, due to the
%
being encoded as%25
.Version
v0.48.05
How did you install?
Docker Compose
To Reproduce
Steps to reproduce the behavior:
posts://example.com/my%20endpoint
Example watch: https://changedetection.io/share/0U5lJwB5bmQa
Expected behavior
The notification should be delivered to the supplied path, e.g.
my%20endpoint
instead ofmy%2520endpoint
.Desktop (please complete the following information):
Additional context
The notification log will show the user-supplied URL (i.e. will not show the problem). This was discovered by wrapping the outbound request in a log statement.
The text was updated successfully, but these errors were encountered: