jira-plugin icon indicating copy to clipboard operation
jira-plugin copied to clipboard

Published 20 hours ago • verified publisher icon jenkinsci

Clean up HTTP authentication code references

Open rantoniuk opened this issue 1 year ago • 5 comments

Clean up HTTP Auth references from the code based on the discussion in https://github.com/jenkinsci/jira-plugin/pull/521#discussion_r1193185025_

rantoniuk avatar May 14 '23 17:05 rantoniuk

Hi. I would like to give this a try, can I work on this?

akshaywakhare avatar May 16 '23 16:05 akshaywakhare

Of course, you're welcome to contribute!

rantoniuk avatar May 17 '23 21:05 rantoniuk

Hey, what should be done with all the (deprecated) functions using useHTTPAuth? Should I update the functions with useBearerAuth or remove them entirely? Sorry if its obvious, I'm new to this.

xRME-18 avatar May 23 '23 21:05 xRME-18

Hi, I looked through the mentioned issue above and concluded the disscussion:

  1. Bearer Authentication is only allowed on Jira Server but not on Jira cloud.
  2. HTTP auth as well as Bearer auth should both exist to meet the requirements of 1.

Hence, I removed the invisibleEntry section of HTTP in the config.jelly file and updated the constructors in file JiraSite.java, as well as the JiraSiteTest.java by adding useBearerAuth. If I'm misunderstanding the issue, please offer more instructions on how I can contribute to this issue, thanks!

Here is my Pull Request: https://github.com/jenkinsci/jira-plugin/pull/562

programbeginnerTW avatar Jul 16 '23 06:07 programbeginnerTW

I would like to work on this issue. Please assign this issue to me.

abhishekshah5486 avatar Jul 10 '24 07:07 abhishekshah5486