Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Make the API return the server public key and add a dedicated route for it" #1024

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

jwr1
Copy link
Member

@jwr1 jwr1 commented Aug 14, 2024

Reverts #1018

I figured out web push messages have the server's public key built into the header of the payload, so a dedicated endpoint to fetch this is unnecessary.

@jwr1 jwr1 added the api API related issues and pull requests label Aug 14, 2024
@jwr1 jwr1 requested a review from BentiGorlich August 14, 2024 17:50
@jwr1 jwr1 enabled auto-merge (squash) August 14, 2024 17:53
@jwr1 jwr1 merged commit 8b52828 into main Aug 14, 2024
7 checks passed
@jwr1 jwr1 deleted the revert-1018-fix/api-missing-push-public-key branch August 14, 2024 18:02
thepaperpilot pushed a commit to thepaperpilot/mbin that referenced this pull request Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api API related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants