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

Update policy to traffic_policy on HTTP/TLS/TCP #890

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

joelhans
Copy link
Contributor

@joelhans joelhans commented Aug 9, 2024

I noticed that after #862, some instances of example config files were changed from policy to traffic_policy, but that hadn't reached the Traffic Policy-specific doc (example for HTTP) for each type of endpoint.

This PR updates those three examples to update the naming. And fixes what I believe to be malformed indentation in the TCP example.

Copy link

vercel bot commented Aug 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ngrok-docs ✅ Ready (Inspect) Visit Preview Aug 9, 2024 7:42pm

@joelhans
Copy link
Contributor Author

joelhans commented Aug 9, 2024

Hi @TheConcierge and @rkolavo! I'm not sure how review requests do/should work here, or if either of you two are the right person to ping here, so just let me know—thanks!

@TheConcierge
Copy link
Contributor

That's my bad. Good catch, Joel!

@joelhans joelhans merged commit b3692d0 into main Aug 9, 2024
3 checks passed
@joelhans joelhans deleted the agent-config-policy branch August 9, 2024 20:20
@joelhans
Copy link
Contributor Author

joelhans commented Aug 9, 2024

@TheConcierge Happy to help! Just noticed it as I was investigating the changelog ahead of working on the next newsletter.

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.

3 participants