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

Redesign protocol code #388

Closed
5 of 8 tasks
snarfed opened this issue Jan 26, 2023 · 1 comment
Closed
5 of 8 tasks

Redesign protocol code #388

snarfed opened this issue Jan 26, 2023 · 1 comment

Comments

@snarfed
Copy link
Owner

snarfed commented Jan 26, 2023 •
edited
Loading

Abstract it like granary abstracts formats. Big project!

Protocols:

Generic API:

  • fetch
  • send
  • receive: only implemented generically, not per protocol
    • maybe implement as big switch statement that calls per-verb handlers (always using AS1 generically)

TODO before deploying or soon after:

  • cache webmention discovery
@snarfed snarfed mentioned this issue Jan 26, 2023
Closed
24 tasks
snarfed added a commit that referenced this issue Mar 9, 2023
@snarfed
start to separate logic from protocols with new Protocol/ActivityPub …
bdb03c3 
…classes

for #388
snarfed added a commit that referenced this issue Mar 9, 2023
@snarfed
abstracting protocols: start to use send()
7080335 
#388
snarfed added a commit that referenced this issue Mar 10, 2023
@snarfed
start to separate logic from protocols with new Protocol/ActivityPub …
ffd8810 
…classes

for #388
snarfed added a commit that referenced this issue Mar 10, 2023
@snarfed
abstracting protocols: start to use send()
3164561 
#388
@snarfed
Copy link
Owner Author

snarfed commented Jun 18, 2023

Subsumed by #512

@snarfed snarfed closed this as not planned Won't fix, can't repro, duplicate, stale Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@snarfed