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

fix: Add expiration date and time for bindings #332

Merged

Conversation

KsaweryZietara
Copy link
Contributor

@KsaweryZietara KsaweryZietara commented Oct 8, 2024

According to OSB API Spec, the Binding Object contains a Metadata Object with expires_at and renew_before which are missing in the current implementation. expires_at was introduces in OSB API 2.16 and renew_before in 2.17.

@blgm
Copy link
Member

blgm commented Oct 9, 2024

Hi @KsaweryZietara. These properties are absolutely in the spec at v2.17, but as far as I'm aware, the main platform that implements OSBAPI - CloudFoundry - only supports up to version v2.15. It might be that my information is out of date, but I just wanted to check that you have a platform that will provide the properties, as it would not help anyone to merge this PR only to find that the platform does not provide the properties anyway.

@KsaweryZietara KsaweryZietara changed the title Add expiration date and time for bindings fix: Add expiration date and time for bindings Oct 9, 2024
@KsaweryZietara
Copy link
Contributor Author

Hi @blgm, the platform in which our broker is registered supports 2.17

@blgm blgm merged commit 1196157 into pivotal-cf:main Oct 9, 2024
4 of 5 checks passed
@KsaweryZietara
Copy link
Contributor Author

Hi @blgm, when will the changes be released?

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