-
Notifications
You must be signed in to change notification settings - Fork 40.8k
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
Management endpoint access and enabled properties are ignored unless the endpoint ID is an exact match #43302
Comments
Sorry, that's a mistake in the docs. It should be I can't reproduce the runtime behavior that you have described. For example, these properties work for me:
The These properties also work for me:
I've tested both with If you would like us to spend some more time investigating, please spend some time providing a complete yet minimal sample that reproduces the problem. You can share it with us by pushing it to a separate repository on GitHub or by zipping it up and attaching it to this issue. |
I seem to have found the problem. The bug occurs depending on the endpoint identifier. That is, if the endpoint identifier was:
However, in version 3.4, only this property is valid:
Another behaviour changed is that in previous version when you have @RestContollerEndpoint (Deprecated):
The endpoint is exposed I think that in this case, 3.4 fixes the previous behaviour. |
Thanks for the sample. I've now reproduced the problem.
We agree. This was raised in #42987 which led to an update to the release notes being made to describe the change. |
I'm migrating to 3.4.0 version and I apreciate that
management.endpoints.enable-by-default
andmanagement.endpoint.id.enabled
are deprecated as it says in the release notes.In these release notes it appears that there are 3 values for the new access properties: none, read-only and unrestricted. However, in the release documentation, the ‘disabled’ is indicated, but disabled is not a valid value:
In my application I have an endpoint running with the following configuration:
However when migrating to version 3.4 this endpoint is no longer exposed.
I thought that being deprecated properties and not deleted, I should not make changes yet.
Anyway I have made the following changes:
and the enpoint is still not exposed.
I have only been able to expose it with the following configuration:
But, this is not the correct behaivor.
The text was updated successfully, but these errors were encountered: