search

jenkinsci / sumologic-publisher-plugin

Jenkins Plugin to send information to Sumo Logic.
https://plugins.jenkins.io/sumologic-publisher/
Apache License 2.0
5 stars 6 forks source link

changes for Security-3117 #36

Closed sumoanema closed 1 year ago

sumoanema commented 1 year ago

These changes are done to fix security issue : https://issues.jenkins.io/browse/SECURITY-3117 The changes are as per the remediation which were suggested in the above issue. After the changes I was able to build the plugin successfully and was able to test the same.

Testing done

Was able to do test connection with the url provided by logging in through a Admin account in jenkins.

### Submitter checklist
- [X] Make sure you are opening from a **topic/feature/bugfix branch** (right side) and not your main branch!
- [X] Ensure that the pull request title represents the desired changelog entry
- [X] Please describe what you did
- [X] Link to relevant issues in GitHub or Jira
- [X] Link to relevant pull requests, esp. upstream and downstream changes
- [X] Ensure you have provided tests - that demonstrates feature works or fixes the issue
sumoanema commented 1 year ago

@Kevin-CB - Hi Kevin. Just a gentle reminder on this.

sumoanema commented 1 year ago

@Kevin-CB - Thanks a lot !!!

I have merged the PR and created a new release - https://github.com/jenkinsci/sumologic-publisher-plugin/releases/tag/sumologic-publisher-2.2.4

Let me know if there is any help required from our side to remove the warning from the plugin.