Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-31612

Stack trace shown with suppress-stack-trace plugin installed

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Environment:
    • Similar Issues:

      Description

      When submitting a form as an unprivileged user, Jenkins reports a permissions exception and also prints a stack trace. We have the suppress-stack-trace plugin installed and expect this stack trace not be printed.

        Attachments

          Issue Links

            Activity

            Hide
            ifernandezcalvo Ivan Fernandez Calvo added a comment - - edited

            tested now all access denied exceptions are controlled.
            I made the PR with the fix.

            Show
            ifernandezcalvo Ivan Fernandez Calvo added a comment - - edited tested now all access denied exceptions are controlled. I made the PR with the fix.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Ivan Fernandez Calvo
            Path:
            src/main/java/org/jenkinsci/plugins/reverse_proxy_auth/ReverseProxySecurityRealm.java
            http://jenkins-ci.org/commit/reverse-proxy-auth-plugin/dc7808b503d4ae2d2e7c74fa35df0821c1b794f0
            Log:
            JENKINS-31612 creates the default filter and adds the new filter to the chain, so maintains the correct filter chain and unwarps access denied exceptions

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ivan Fernandez Calvo Path: src/main/java/org/jenkinsci/plugins/reverse_proxy_auth/ReverseProxySecurityRealm.java http://jenkins-ci.org/commit/reverse-proxy-auth-plugin/dc7808b503d4ae2d2e7c74fa35df0821c1b794f0 Log: JENKINS-31612 creates the default filter and adds the new filter to the chain, so maintains the correct filter chain and unwarps access denied exceptions
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Wilder Rodrigues
            Path:
            src/main/java/org/jenkinsci/plugins/reverse_proxy_auth/ReverseProxySecurityRealm.java
            http://jenkins-ci.org/commit/reverse-proxy-auth-plugin/cfa947cdc738e3ef992e99089bd207b12aaa4480
            Log:
            Merge pull request #28 from kuisathaverat/JENKINS-31612

            JENKINS-31612 Fix filter chain

            Compare: https://github.com/jenkinsci/reverse-proxy-auth-plugin/compare/f31a7875d611...cfa947cdc738

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Wilder Rodrigues Path: src/main/java/org/jenkinsci/plugins/reverse_proxy_auth/ReverseProxySecurityRealm.java http://jenkins-ci.org/commit/reverse-proxy-auth-plugin/cfa947cdc738e3ef992e99089bd207b12aaa4480 Log: Merge pull request #28 from kuisathaverat/ JENKINS-31612 JENKINS-31612 Fix filter chain Compare: https://github.com/jenkinsci/reverse-proxy-auth-plugin/compare/f31a7875d611...cfa947cdc738
            Hide
            jec Josh Cook added a comment -

            I see that changes to address have been merged into the suppress-stack-trace plugin master branch, but there hasn't been a subsequent release.

            Is there any ETA for when a new version of this plugin will be available including these changes?

            Show
            jec Josh Cook added a comment - I see that changes to address have been merged into the suppress-stack-trace plugin master branch, but there hasn't been a subsequent release. Is there any ETA for when a new version of this plugin will be available including these changes?
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            It has been actually released in 1.6.0

            Show
            oleg_nenashev Oleg Nenashev added a comment - It has been actually released in 1.6.0

              People

              • Assignee:
                ifernandezcalvo Ivan Fernandez Calvo
                Reporter:
                jec Josh Cook
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: