matrix-org / sygnal

Sygnal: reference Push Gateway for Matrix
Apache License 2.0
153 stars 139 forks source link

Allow to configure if pushkey should be converted to hex before sending to APNs #344

Closed danbim closed 7 months ago

danbim commented 10 months ago

I ran into the same problem as https://github.com/matrix-org/sygnal/issues/301 and decided to instead make this configurable in Sygnal. Not sure if this is the right approach, happy for feedback and adopting to something else :)

Signed-off-by: Daniel Bimschas daniel@bimschas.com / daniel.bimschas@inoio.de

clokep commented 7 months ago

Would you mind adding a newsfile?

clokep commented 7 months ago

@danbim Ah, sorry can you sign-off on your changes? Should be good to go if CI passes! 🎉

danbim commented 7 months ago

@clokep Haven't used this mechanism before. Should I rewrite history for this or is adding an empty commit with sign off sufficient?

clokep commented 7 months ago

@clokep Haven't used this mechanism before. Should I rewrite history for this or is adding an empty commit with sign off sufficient?

The easiest thing is usually to edit the PR description and manually add the text. Adding an empty commit to the PR is fine too.

danbim commented 7 months ago

@clokep Haven't used this mechanism before. Should I rewrite history for this or is adding an empty commit with sign off sufficient?

The easiest thing is usually to edit the PR description and manually add the text. Adding an empty commit to the PR is fine too.

Done. Added my private email as the commits were accidentally signed with my old employers email address 🙈

clokep commented 7 months ago

Thank you!

danbim commented 7 months ago

@clokep Out of egoistic personal interest to remove my patched fork to simplify my deployment: any plans when to have a new release including this MR? :)

clokep commented 7 months ago

Will need to ask @matrix-org/synapse-core id they have another release planned soon.

danbim commented 5 months ago

@clokep Do you maybe have any update on the question of a release? I have seen this project might move to element-hq due to the license change but so far I can't find it there. Cheers!

clokep commented 5 months ago

@danbim You'll need to ask the current team -- maybe @erikjohnston or @reivilibre would know.

Half-Shot commented 3 months ago

This appears to have now happened -> https://github.com/matrix-org/sygnal/releases/tag/v0.14.0