-
Notifications
You must be signed in to change notification settings - Fork 429
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
[atlassian_jira] Ensure from/to timestamps are properly encoded #7759
Conversation
- Ensure that from/to timestamps are properly encoded as positive (+) timezone offsets may be interpreted as spaces by the other end (Jira Cloud).
🌐 Coverage report
|
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you modify the system test such that it would fail without this fix?
packages/atlassian_jira/data_stream/audit/agent/stream/httpjson.yml.hbs
Outdated
Show resolved
Hide resolved
Can you please also inspect the atlassian_confluence integration to see if it suffers from a similar lack of url-encoding, and if so open an issue. |
At a quick glance, looks like it uses the same methods, so very likely suffers from the same problem. I can open a new issue. |
- Set pagination parameters via set transforms - Add regex validation for from/to timestamps in system tests - Add hit count asserts for system tests
I added regex and hit counts to validate the tests. Adding anchors to the regex didn't seem to work and I don't know why. I'm not entirely sure what string it is testing against, but I do know it's after it has been decoded (i.e., I have to test for |
packages/atlassian_jira/data_stream/audit/agent/stream/httpjson.yml.hbs
Outdated
Show resolved
Hide resolved
Package atlassian_jira - 1.16.1 containing this change is available at https://epr.elastic.co/search?package=atlassian_jira |
What does this PR do?
Checklist
changelog.yml
file.How to test this PR locally
elastic-package test