-
Notifications
You must be signed in to change notification settings - Fork 407
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
"BootstrapConfig" should be a list of request #437
Comments
After several thoughts and tries about making the We try to implement this in #883 and as explained at #883 (comment) this bring new issues ... Those makes us think that maybe we should keep
In addition, the code should be refactored to make easier to integrate custom behavior and so support all cases. This means that recent addition of #883 was probably a bad idea and will be removed/replaced by new classes. |
#1027 is a new try to achieve those goals. |
After more thoughts, we decide that this was not the good approach. See #437 (comment) for more details. Consequences BootstrapConfig is no more deprecated.
After more thoughts, we decide that this was not the good approach. See #437 (comment) for more details. Consequences BootstrapConfig is no more deprecated.
I think #1027 achieve this goal. |
Currently the bootstrap server is configurable by adding a bootstrapConfig for a given endpoint.
This bootstrap config is a list of
Security
objects andServer
objects.A bootstrap session will start by a
Delete
on/
then aWrite
request for eachSecurity
objects then aWrite
request for eachServer
objects.This is not so flexible.
The bootstrapConfig should rather be a list of requests (write,delete) to send.
This allow to :
But this will not cover cases where we need to make complex processing (using discover, using previous result to guess what to send ... ... ), I don't know what this might look like.
The text was updated successfully, but these errors were encountered: