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

Jump to bottom

System subject #195

Open
wants to merge 81 commits into
base: main
Choose a base branch
from
Open

System subject #195

wants to merge 81 commits into from

Conversation

cwperks
Copy link
Owner

@cwperks cwperks commented Sep 19, 2024

Description

WIP To add a SystemSubject and replace usages of threadContext.stashContext within OpenSearch core

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

cwperks added 30 commits May 30, 2024 16:33
@cwperks
Create ExecutionContext and show example with ActionPluginProxy
3506973 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Only allow core to set the ExecutionContext
f8cf238 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into plugin-aware-thread-context
19155ca
@cwperks
WIP on plugin aware thread context
b6b2a19 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Plugin Aware API Handling
8165f05 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into plugin-aware-thread-context
7e603e1
@cwperks
Add test to verify that ExecutionContext is being populated during Re…
5b989d6 
…stHandling

Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into plugin-aware-thread-context
444fde7
@cwperks
Merge branch 'main' into plugin-aware-thread-context
aaba063
@cwperks
Clear context in a finally block
eec8b96 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Create switchContext method in ThreadContext and make pluginExecution…
8a98618 
…Stack a stack

Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into plugin-aware-thread-context
5f7a26f
@cwperks
WIP on plugin aware stash context
6b6e955 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Create class called PluginAwareNodeClient that provides a method call…
73835ed 
…ed switchContext

Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Remove ExecutionContext class
4d048b7 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Update javadoc
70c0c80 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Change createComponents to take in PluginAwareNodeClient
dc5c5a8 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Update all instances of createComponents
e26ac6c 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Initialize clients
c2e0a64 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into plugin-aware-thread-context
1b234b5
@cwperks
Remove casting
1d7df43 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
WIP on notion of ContextSwitcher
711d542 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Make stashContext package-private
e2ebb36 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Make markAsSystemContext package-private
ed71247 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into plugin-aware-thread-context
4b27f3a
@cwperks
Add javadoc on param
b031db9 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Remove SystemContextSwitcher
54439a0 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into plugin-aware-thread-context
383fa83
@cwperks
Merge with main
dd8e7c3 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Cleanup
2ff8f86 
Signed-off-by: Craig Perkins <[email protected]>
cwperks added 30 commits August 15, 2024 16:38
@cwperks
Merge branch 'main' into plugin-aware-thread-context
8f38206
@cwperks
Simplify PR, make NoopPluginSubject and introduce IdentityAwarePlugin
16f4251 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Add final
7b3b5ec 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Remove server dependency
0a26f26 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Remove AbstractSubject
94a5f2c 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Remove unnecessary changes
19dffb7 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Add javadoc to NoopPluginSubject
17abb63 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into plugin-aware-thread-context
d6be989
@cwperks
Rename to assignSubject
48bd43c 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Add experimental label
be9a89a 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Add getPluginSubject(plugin) to IdentityPlugin
ca1f297 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Make runAs generic
84c0cc3 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
package-private constructor
c200272 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Move IdentityAwarePlugin initialization
be9b3cc 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Create separate PluginSubject interface
9c07676 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Remove authenticate method
901d53d 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Remove import
55834df 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Separate UserSubject and PluginSubject
7374479 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Terminate TestThreadPool
efac919 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
mock ThreadPool in RestSendToExtensionActionTests
1536380 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Fix Thread leak
e6eb37e 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Add to CHANGELOG
2c9ba56 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Rename to getCurrentSubject
9c5b2c4 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Add type check
4b8c7bd 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
WIP on system subject
0531871 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'plugin-aware-thread-context' into system-subject
9e30148
@cwperks
Introduce SystemSubject and replace usages of stashContext
29e3527 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Merge branch 'main' into system-subject
18737dc
@cwperks
Exception handling
4f4d758 
Signed-off-by: Craig Perkins <[email protected]>
@cwperks
Add tests for SystemSubject
6702707 
Signed-off-by: Craig Perkins <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cwperks