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.

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

Finish adding Event Grid validation, and add local testing tools #48

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

bcspragu
Copy link
Collaborator

This PR finishes up the work of validating requests from Event Grid, and adds support for running the server with a public-facing endpoint via the --with_public_endpoint=<subdomain> flag, which is proxied via frp.

I didn't use our frpembed library because there's an issue with quic-go not supporting Go 1.20 (may be fixed, didn't want to go down the rabbithole)

This PR finishes up the work of validating requests from Event Grid, and adds support for running the server with a public-facing endpoint via the `--with_public_endpoint=<subdomain>` flag, which is proxied via [frp](https://github.com/fatedier/frp).

I didn't use our [frpembed](https://github.com/Silicon-Ally/frpembed) library because there's an issue with `quic-go` not supporting Go 1.20 (may be fixed, didn't want to go down the rabbithole)
@bcspragu bcspragu requested a review from gbdubs October 31, 2023 21:41
@bcspragu bcspragu merged commit dac238e into main Oct 31, 2023
2 checks passed
@bcspragu bcspragu deleted the brandon/event-grid branch October 31, 2023 21:41
Copy link
Contributor

@gbdubs gbdubs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG - looking forward to using.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants