search

jenkinsci / build-monitor-plugin

Jenkins Build Monitor Plugin
https://plugins.jenkins.io/build-monitor-plugin/
MIT License
727 stars 280 forks source link

Remove old Jenkins 1.x .crumb header #303

Open fredrikand opened 7 years ago

fredrikand commented 7 years ago

There are still some leftovers using the old .crumb header in the java scripts. I am not sure what effect it would have if they were removed. But it seems like they should be possible to remove for Jenkins 2.x installations?

Reason for bothering about the .crumb header is annoying logs in Nginx. :-)

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/43973249-remove-old-jenkins-1-x-crumb-header?utm_campaign=plugin&utm_content=tracker%2F580790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F580790&utm_medium=issues&utm_source=github).
jan-molak commented 7 years ago

Thanks for pointing this out @fredrikand - since Build Monitor is still predominantly being used on Jenkins 1.x I guess we'd need to issue a different crumb header depending on the installation.

Would you be willing to submit a PR?