search

fabric8-services / fabric8-jenkins-proxy

Apache License 2.0
1 stars 16 forks source link

[DNM] Issue #274 Check cookies first #275

Closed kishansagathiya closed 6 years ago

kishansagathiya commented 6 years ago

While proxying using UI, first check if we find JSESSIONID cookie or JenkinsIdled cookie and if we don't then look for token_json in the URL

Fixes #274

alien-ike commented 6 years ago

Ike Plugins (test-keeper)

Thank you @kishansagathiya for this contribution!

It appears that no tests have been added or updated in this PR.

Automated tests give us confidence in shipping reliable software. Please add some as part of this change.

If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.

For more information please head over to official documentation. You can find there how to configure the plugin - for example exclude certain file types so if PR contains only them it won't be checked.

kishansagathiya commented 6 years ago

Need to check if this is breaking anything