-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Security in Core] Core licensing service #187296
Comments
Pinging @elastic/kibana-core (Team:Core) |
Pinging @elastic/kibana-security (Team:Security) |
Pinging the security team for visibility |
I will try to summarize, please correct me if anything is wrong or incomplete: So, the primary purpose of moving the licensing APIs to Core is that it's a prerequisite to move parts of the security plugin's implementation into Core. ATM, for the parts of the security APIs we're exposing from Core, we're using a registration pattern, so that the security plugin registers its implementation during I however doubt that we will be able to follow the same patterns for the licensing plugin / API, especially given security depends on it. Having the same delegation / registration at So the more I think about it, the more I believe we will have no other choice than effectively moving the licensing plugin's implementation into Core (or at least utilizing it directly, it could also be done by moving the implementation in some package that Core would import). The question emerging from there is simply the... licensing of licensing (funny, right?) Atm
And I'm not sure who would to able to take the decision on either of those options. @lukeelmers do you have any idea? |
Part of this is a legal/product question that I'm also wondering about in relation to our sustainable architecture initiative -- I will dig into this further and see if I can get some clarity on the consequences of moving some single-licensed code to being dual-licensed. |
@pgayvallet @TinaHeiligers Okay, after doing some digging, I can confirm that any code which requires non-free (Gold+) licenses must remain single-licensed under ELv2. We can discuss more about the "why" internally, but I think this means that given the current structure of the repo, we'd need to keep the That said, if we were to revisit the structure of the repository as we have been discussing and handle our license differentiation without x-pack, it may be possible to still have this live as a part of core in the long-run. (e.g. if we did something like |
Yeah, this is our most viable and realistic option now (we will likely need it for security too) |
@pgayvallet for all of security? If so, we'll need to figure out what to do with |
For anything implementation-related (so any concrete code) that currently lives in the |
I propose closing this given the reason in #187296 (comment). We can revisit it at a later stage. |
Reopening to discuss if we could expose the apis from the licensing lifetime contracts is possible and if that is enough for now It would be similar to how we expose @SiddharthMantri Let me know if you want to chat more about the details. |
We discussed if there's any benefit for plugin developers in exposing the licensing APIs from core and there isn't. |
Part of #186574
The security plugin depends on the licensing service.
Once security implementations move to Core, this dependency won't be possible.
Moving licensing to Core would solve this, however, security is under the proprietary license, while Core is under the dual license.
Scope:
The text was updated successfully, but these errors were encountered: