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.

Sign up for GitHub

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

Jump to bottom

feat: add support for Pushy push notifications #254

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

feat: add support for Pushy push notifications #254

wants to merge 37 commits into from

Conversation

jmhossler
Copy link
Contributor

This adds basic support for Pushy notifications.

dballance and others added 30 commits April 1, 2024 12:05
@dballance @jmhossler
fix: move token generation to agent (#1)
ce1eff4
@jmhossler
chore: undo random formatting changes that aren't necessary
ae61126
@jmhossler
feat: implement a pushy push notification adapter
757e948
@jmhossler
fix: implement connect_socket method
bb265f9
@jmhossler
feat: add pushy notification constructor
2aa8eee
@jmhossler
fix: do @SPEC new instead of @new smh
4c3abe2
@jmhossler
fix: add return type to new spec
917711f
@jmhossler
fix: update connect parameters and add docs link in mix
b0f48c6
@jmhossler
debug: add error log so I can tell why connection to pushy isn't working
fa4b59d
@jmhossler
fix: require logger in pushy dispatcher
addf2ae
@jmhossler
fix: do a regular inspect of the error instead when connecting to pushy
f4c279c
@jmhossler
fix: adjust socket connection settings to attempt to resolve handshak…
a39adde 
…e failure
@jmhossler
refactor: don't use socket when connecting to pushy, just make http r…
ff3ed8a 
…equests since ssl cert was throwing
@jmhossler
fix: correctly call pushy_headers and remove unnecessary match
3c89181
@jmhossler
fix: use notification encode_requests to encode the payload
d725ba5
@jmhossler
fix: move over encoding logic so I can encode properly
73cea75
@jmhossler
fix: remove unmatched curly brace and use correct encode request impl
e42caab
@jmhossler
fix: don't pass in more params than necessary to error.parse
839ac03
@jmhossler
fix: correct the pushy path for pushing notifications
c062ec2
@jmhossler
fix: remove error parse on good data, just return response from pushy
f7230b1
@jmhossler
fix: pass notification around appropriately in do_push for pushy
a7d40d8
@jmhossler
feat: add result parser to handle both error and success cases
be63a72
@jmhossler
fix: add additional notification fields to hold response info
1f8985b
@jmhossler
fix: address some invalid variable refs and remove unused param from …
c9ec79a 
…error
@jmhossler
fix: pass the resulting json instead of some fictitious result parser…
0884f66 
… to the result parser
@jmhossler
fix: handle case where failed devices might not be included
83bf249
@jmhossler
fix: remove old reference to failed devices
29fba72
@jmhossler
fix: parse errors into notification structs instead of just atom
5b19e6d
@jmhossler
feat: add helper methods for filling in additional parameters for pus…
bf90370 
…hy notifications
@jmhossler
fix: add missing empty curly bracket and format files
7af0b96
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jmhossler @dballance @nathanalderson