jenkinsci / office-365-connector-plugin

Office 365 Connector plugin sends jobs status notifications to Microsoft Teams or Outlook
https://plugins.jenkins.io/Office-365-Connector/
Apache License 2.0
89 stars 86 forks source link

Feat/Support for local plugin proxy #277

Open rockleez opened 2 years ago

rockleez commented 2 years ago

Summary This PR addresses JENKINS-61989 and implements proxy support on the Office365ConnectorPlugin itself instead of utilizing the Jenkins global proxy config. In my case i would have to add multiple no proxy hosts to achieve the existing stability of the Jenkins server.

Implementation

image

Testing Unit Tests

Integration Tests

Please roll this into a new release once approved

rockleez commented 2 years ago

@damianszczepanik on a similar not I've noticed that the proxy config only takes effect when the job config is opened and saved even with no changes being made. Is this normal behavior due to the architecture of Jenkins?

Example.

rockleez commented 2 years ago

@damianszczepanik do you have an eta on when this review can be completed and merged into a release. Thanks

damianszczepanik commented 2 years ago

Yes, saving updates job

damianszczepanik commented 1 year ago

Github says that this still has some conflicts