-
Notifications
You must be signed in to change notification settings - Fork 58
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
fix(APIController): Allow self-test depending on token and not user-agent #2175
Conversation
/backport to stable31 |
🐢 Performance warning. |
7e857a4
to
1634fdc
Compare
🐢 Performance warning. |
Signed-off-by: provokateurin <[email protected]>
1634fdc
to
4c80910
Compare
🐢 Performance warning. |
The check was incomplete … It would have needed to extend to check if it's actually a permanent token and from there we should then have checked |
I would also like to use this endpoint without having a matching user-agent.
The token is a better method to verify the client can even handle push notifications, because in order to have a valid token it already needs to be registered for push notifications.