Dry Out Jira-V2 Output - #1333
Closed
gavinelder wants to merge 4 commits into
Closed
Conversation
chunyong-lin
approved these changes
Jul 20, 2022
chunyong-lin
left a comment
Contributor
There was a problem hiding this comment.
It is lots of copy and paste 😄 Thanks for the contribution.
Side note, we probably can write a new output based on JIRA's official python package jira, maybe name it jira-cloud or jira-saas. I actually use it for other projects and works well.
Contributor
Author
|
@chunyong-lin |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
to: @ryandeivert @chunyong-lin @Ryxias
cc : @blakemotl
related to: #1311
Background
The following change was discussed on #1311 where the original Jira-v2 was just a heavy cut/paste job with some slight modifications, which is not ideal.
https://streamalert.slack.com/archives/C3BHE2Z0S/p1610399094191300
This is cleaning up all of that tech debt.
Changes
_api_keyis nowpasswordand we use the existing credentials helper.Contentious points
api_keyhas changed topassword, I can add a small helper to pull from this field ifpasswordis none etc but curious thoughts on that one.self._cookiewhich is now holding the base64 encoded authorization bearer token.Testing
This was tested in conjunction with #1311 against a live environment.