Skip to content

fix: support proxy-safe subscriptions delete url#10

Merged
Priveetee merged 1 commit intomainfrom
dev
Apr 13, 2026
Merged

fix: support proxy-safe subscriptions delete url#10
Priveetee merged 1 commit intomainfrom
dev

Conversation

@Priveetee
Copy link
Copy Markdown
Owner

Summary

  • add a proxy-safe delete path for subscriptions via DELETE /subscriptions?url=...
  • make DELETE /subscriptions/{channelUrl...} resolve url from query first, then robustly decode path tail
  • keep existing behavior while avoiding fragile segment reconstruction of encoded channel URLs
  • add integration tests covering query delete and proxy-decoded path fallback

Why

Behind nginx, encoded subscription URLs can be decoded/normalized in a way that breaks https:// path reconstruction and causes false 404 on unsubscribe.

Validation

  • full suite: ./gradlew test -> success
  • new tests: SubscriptionsDeleteProxyRoutesTest

@Priveetee Priveetee merged commit 818fd54 into main Apr 13, 2026
5 checks passed
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.

1 participant