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.

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

Dry Out Jira-V2 Output #1333

Closed
wants to merge 4 commits into from
Closed

Conversation

gavinelder
Copy link
Contributor

@gavinelder gavinelder commented Jul 19, 2022

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

  • Dry Out the config.
  • _api_key is now password and we use the existing credentials helper.

Contentious points

  • This will break existing Jira-V2 users as the credential for api_key has changed to password , I can add a small helper to pull from this field if password is none etc but curious thoughts on that one.
  • This overloads self._cookie which is now holding the base64 encoded authorization bearer token.

Testing

This was tested in conjunction with #1311 against a live environment.

Copy link
Contributor

@chunyong-lin chunyong-lin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@gavinelder
Copy link
Contributor Author

@chunyong-lin jira-cloud SGTM I would like to see what #1311 and v4 look like before a major overhaul, however building and introducing before a major breaking change and removing old Outputs like PD v1, Jira, etc may make sense.

@gavinelder gavinelder closed this Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants