Skip to content

Exposing basics of ResourceDomainConfiguration via Java API - #4335

Merged
jglick merged 2 commits into
jenkinsci:masterfrom
jglick:expose-ResourceDomainConfiguration
Nov 1, 2019
Merged

Exposing basics of ResourceDomainConfiguration via Java API#4335
jglick merged 2 commits into
jenkinsci:masterfrom
jglick:expose-ResourceDomainConfiguration

Conversation

@jglick

@jglick jglick commented Oct 31, 2019

Copy link
Copy Markdown
Member

Amends #4239. I needed to get and set the URL here from Java code, but ran into a compilation error. Note that these methods are already effectively publicized via reflection in JCasC.

(Note that my real goal is simply to define the URL, or the host portion of the URL, via system property, without a configuration-as-code dependency or the like. Making these Java calls via @Initializer is just one way to accomplish that.)

Proposed changelog entries

  • Making some methods in ResourceDomainConfiguration accessible from plugins.

@jglick jglick added the developer Changes which impact plugin developers label Oct 31, 2019
@jglick
jglick requested a review from daniel-beck October 31, 2019 17:21
Comment thread core/src/main/java/jenkins/security/ResourceDomainConfiguration.java Outdated

@daniel-beck daniel-beck 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.

Would it make sense to mark this as @Beta or some similar indicator that consumers should not rely on API stability?

Comment thread core/src/main/java/jenkins/security/ResourceDomainConfiguration.java Outdated
@jglick

jglick commented Oct 31, 2019

Copy link
Copy Markdown
Member Author

Would it make sense to mark this as @Beta

Good idea, I will do that.

@jglick
jglick requested a review from daniel-beck October 31, 2019 18:21
@jglick
jglick merged commit dea8908 into jenkinsci:master Nov 1, 2019
@jglick
jglick deleted the expose-ResourceDomainConfiguration branch November 1, 2019 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer Changes which impact plugin developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants