Skip to content

Fix Edge pushes: TTL configurable #773

Open
goinnn wants to merge 2 commits into
jazzband:masterfrom
we-are-Joinup:fix-edge-pushes-add-ttl
Open

Fix Edge pushes: TTL configurable #773
goinnn wants to merge 2 commits into
jazzband:masterfrom
we-are-Joinup:fix-edge-pushes-add-ttl

Conversation

@goinnn

@goinnn goinnn commented Jun 3, 2025

Copy link
Copy Markdown
Member

This implementation makes the TTL configurable A TTL is required for Edge; without it, a 400 Bad Request error is returned.

…Edge; without it, a 400 Bad Request error is returned.
@goinnn goinnn changed the title This implementation makes the TTL configurable Fix Edge pushes: TTL configurable Jun 3, 2025
Comment thread README.rst

@50-Course 50-Course left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is good to go!

@50-Course

Copy link
Copy Markdown
Member

just realize we still don't have second approval on this, please take a look at this too, if you can, let's merge this in 🙏🏼

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