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(mymdc): add proxying to mymdc endpoints #11

Merged
merged 44 commits into from
Sep 24, 2024
Merged

feat(mymdc): add proxying to mymdc endpoints #11

merged 44 commits into from
Sep 24, 2024

Conversation

RobertRosca
Copy link
Member

@RobertRosca RobertRosca commented Sep 24, 2024 •
edited
Loading

feat:

  • add api routes to proxy through for clients
  • add token writer service

refactor:

  • improve proxy path behaviour

@RobertRosca RobertRosca marked this pull request as draft September 24, 2024 07:08
@RobertRosca RobertRosca changed the title Feat/mymdc proxy feat(mymdc): add proxying to mymdc endpoints Sep 24, 2024
@RobertRosca RobertRosca force-pushed the feat/web-interface-message-history branch from 11452da to dc42c4d Compare September 24, 2024 07:15
Base automatically changed from feat/web-interface-message-history to main September 24, 2024 07:18
@RobertRosca
build(docker): use string labels in compose files for docker stack co…
99e5fe4 
…mpatibility
@RobertRosca
build(docker): simplify docker command calls for watch, staging, prod
7a4c305
@RobertRosca
feat(api): add initial mymdc proxy functionality
fa294db
@RobertRosca
feat(api): add zulip tags to zulip proxy endpoints
12131fb
@RobertRosca
feat: improve logging for initial trusted hosts/traefik setup
5674140
@RobertRosca
fix(api): get path from scope to unify behaviour behind proxy
a318987
@RobertRosca
revert: build docker compose changes
ad02a08
@RobertRosca
chore: linting, type, warning fixes
e91e26a
@RobertRosca
build: update lockfile
3edde88
@RobertRosca
feat(api): split out mymdc endpoints, refactor
e88706a
@RobertRosca
feat(mymdc): improve error handling for responses
71570b6
@RobertRosca
feat(mymdc): add get proposal id method
ecca5c7
@RobertRosca
feat(models): add proposal id to client model
8606ca1
@RobertRosca
revert: refactor mymdc api router
2def2a5
@RobertRosca
feat(api): adjust mymdc response check order
f18b1ce
@RobertRosca
feat(api): pass through page size/no for mymdc proxy call
f096128
@RobertRosca
feat(api): refactor mymdc proxy, more verbose but clearer flows
f385b07
@RobertRosca
feat(repositories): error on insert if item exists
97c86d0
@RobertRosca
chore: add migration to gitignore
f368b4b
@RobertRosca
fix(mymdc): fix parsing result to dict/json
abd3184
@RobertRosca
feat(api): add token write endpoint and service
1e86ed6
@RobertRosca
chore(repositories): allow int type for key in get
9ae9e05
@RobertRosca
revert: feat add token write endpoint
5519909
@RobertRosca
chore: add id_ed25519 to gitignore
0ceae69
@RobertRosca
fix(build): traefik path prefix only accept with slash or nothing
bc6ebad 
fixes issue with path prefix matching where /zwop also matches /zwop-staging
@RobertRosca
fix(build): update develop watch docker compose file path
8547978
@RobertRosca
fix(build): update network to proxy swarm
ceb9bc6
@RobertRosca
fix(services/write_tokens): store and use ssh known hosts
13a85fe
@RobertRosca
fix(services/write_tokens): use correct paths and permissions for tokens
e96e17f 
zulip.cfg should have global read/write

mymdc-credentials.yml should not allow other read/write
@RobertRosca
feat(services/write_tokens): note if client was created in details
7dfc585
@RobertRosca
chore(services/write_tokens): fix type annotation errors
1a45b2b
@RobertRosca
feat(mymdc): reraise error in mymdc response w/ custom exception
48ed3c6
@RobertRosca
feat(models/client): make zulip bot config optional
87fd4dd
@RobertRosca
feat(api): try to get bot config if bot missing for client on calls t…
1854713 
…o zulip endpoints
@RobertRosca
feat(services): catch and handle exceptions related to missing bot co…
b0f1708 
…nfig
@RobertRosca
feat(deploy): use proxy network instead of proxy-swarm
416bb0b
@RobertRosca
feat(deploy): add restart policy
19d977c
@RobertRosca
fix(deploy): update healthcheck url
7a4f768
@RobertRosca
fix(deploy): add build flag to docker task
68392fc
@RobertRosca
fix(services/mymdc): remove error string check for no mymdc logbook
648b8e5
@RobertRosca
fix(deploy): update Taskfile to use compose not stack
d9f2ee6
@RobertRosca
fix(deploy): remove escaped pathprefix backslash
8f8a785
@RobertRosca
fix(deploy): add up -d to prod and staging tasks
0bd3375
@RobertRosca
fix(deploy): set writer zwop url for staging
cb80eb1
@RobertRosca RobertRosca marked this pull request as ready for review September 24, 2024 07:21
@RobertRosca RobertRosca merged commit 5764230 into main Sep 24, 2024
1 of 3 checks passed
@RobertRosca RobertRosca deleted the feat/mymdc-proxy branch September 24, 2024 07:21
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.
1 participant
@RobertRosca