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

GET MAY return creds unable to be used #517

Merged
merged 15 commits into from
Jul 12, 2018
Merged

GET MAY return creds unable to be used #517

Changes from 10 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
e755cf2
GET may return creds unable to be used
MHBauer May 15, 2018
a859d31
all or nothing on credentials.
MHBauer May 15, 2018
94beff6
proposed wording that addresses asynchronous bind
MHBauer May 23, 2018
d3f3d3d
add same text to binding create
MHBauer Jul 3, 2018
f296861
accidental word wrapping
MHBauer Jul 3, 2018
0094af7
accidental deletion
MHBauer Jul 3, 2018
3868de4
Merge branch 'master' into get-no-creds
MHBauer Jul 12, 2018
430cea4
add to split response?
MHBauer Jul 12, 2018
41513fd
oops.
MHBauer Jul 12, 2018
5979f6e
move to new location
MHBauer Jul 12, 2018
9f70b38
make sure we're in a GET response
MHBauer Jul 12, 2018
67bdcd8
dug's suggested wording
MHBauer Jul 12, 2018
2fb4952
remove old text
MHBauer Jul 12, 2018
c86b130
move to overall binding block
MHBauer Jul 12, 2018
2ad9c34
hard 80 col wrap
MHBauer Jul 12, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions spec.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1195,6 +1195,12 @@ utilize the Service Instance. Credentials SHOULD be unique whenever possible, so
access can be revoked for each binding without affecting consumers of other
bindings for the Service Instance.

This field MAY be omitted if the Service Broker either cannot return the credentials
or does not want to for security reasons. During an asynchronous bind
operation the Service Broker MUST include this field in at least the first response
Copy link
Contributor

Choose a reason for hiding this comment

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

first GET response (with link)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

link to what?

after the Service Binding has been successfully created, otherwise the Platform will
not have the credentials to pass along.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dug suggested

Note: some brokers (due to security concerns... blah blah) MAY choose to only return the credentials for a binding once even though there could be replayed binding requests or GETs of the binding resource.

#### Log Drain

There are a class of Service Offerings that provide aggregation, indexing, and
Expand Down