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

Environment properties stopped working in LDAP 1.16

    Details

    • Similar Issues:

      Description

      Under Advanced Configuration for the LDAP plugin, adding new Environment Properties does not apply to config.

      After I save, the configuration I added for environment properties aren't found in the config.xml, and going back to the LDAP plugin settings, the config isn't there as well.

       

      This worked fine in previous LDAP version 1.15. But after the 1.16, the settings in the GUI has been updated, I guess something was missed about the environment properties.

        Attachments

          Issue Links

            Activity

            Hide
            rsandell rsandell added a comment -

            1.17

            Show
            rsandell rsandell added a comment - 1.17
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Robert Sandell
            Path:
            src/main/java/hudson/security/LDAPSecurityRealm.java
            src/main/java/jenkins/security/plugins/ldap/LDAPConfiguration.java
            src/main/resources/hudson/security/LDAPSecurityRealm/config.jelly
            src/main/resources/jenkins/security/plugins/ldap/LDAPConfiguration/config.jelly
            src/test/java/hudson/security/LDAPEmbeddedTest.java
            src/test/java/hudson/security/LDAPSecurityRealmTest.java
            http://jenkins-ci.org/commit/ldap-plugin/d181e7f505b6bc1aef75cca34e81f5f8cca3c87c
            Log:
            Merge pull request #22 from rsandell/JENKINS-45431

            [Fix JENKINS-45431] Moved the config snippet to the correct jelly

            Compare: https://github.com/jenkinsci/ldap-plugin/compare/fae415f0f20b...d181e7f505b6

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Robert Sandell Path: src/main/java/hudson/security/LDAPSecurityRealm.java src/main/java/jenkins/security/plugins/ldap/LDAPConfiguration.java src/main/resources/hudson/security/LDAPSecurityRealm/config.jelly src/main/resources/jenkins/security/plugins/ldap/LDAPConfiguration/config.jelly src/test/java/hudson/security/LDAPEmbeddedTest.java src/test/java/hudson/security/LDAPSecurityRealmTest.java http://jenkins-ci.org/commit/ldap-plugin/d181e7f505b6bc1aef75cca34e81f5f8cca3c87c Log: Merge pull request #22 from rsandell/ JENKINS-45431 [Fix JENKINS-45431] Moved the config snippet to the correct jelly Compare: https://github.com/jenkinsci/ldap-plugin/compare/fae415f0f20b...d181e7f505b6
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: rsandell
            Path:
            src/main/resources/hudson/security/LDAPSecurityRealm/config.jelly
            src/main/resources/jenkins/security/plugins/ldap/LDAPConfiguration/config.jelly
            src/test/java/hudson/security/LDAPSecurityRealmTest.java
            http://jenkins-ci.org/commit/ldap-plugin/04ce673f78083d4521ce058d549b8fb91ae7fdc2
            Log:
            [Fix JENKINS-45431] Moved the config snippet to the correct jelly

            Missed when implementing multi OU support

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: rsandell Path: src/main/resources/hudson/security/LDAPSecurityRealm/config.jelly src/main/resources/jenkins/security/plugins/ldap/LDAPConfiguration/config.jelly src/test/java/hudson/security/LDAPSecurityRealmTest.java http://jenkins-ci.org/commit/ldap-plugin/04ce673f78083d4521ce058d549b8fb91ae7fdc2 Log: [Fix JENKINS-45431] Moved the config snippet to the correct jelly Missed when implementing multi OU support
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: rsandell
            Path:
            src/test/java/hudson/security/LDAPEmbeddedTest.java
            src/test/java/hudson/security/LDAPSecurityRealmTest.java
            http://jenkins-ci.org/commit/ldap-plugin/93e23f72d987c6f38639d20cbc8aaca1206c79f4
            Log:
            Merge branch 'JENKINS-45431' of https://github.com/fcojfernandez/ldap-plugin into JENKINS-45431

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: rsandell Path: src/test/java/hudson/security/LDAPEmbeddedTest.java src/test/java/hudson/security/LDAPSecurityRealmTest.java http://jenkins-ci.org/commit/ldap-plugin/93e23f72d987c6f38639d20cbc8aaca1206c79f4 Log: Merge branch ' JENKINS-45431 ' of https://github.com/fcojfernandez/ldap-plugin into JENKINS-45431
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Francisco Javier Fernandez Gonzalez
            Path:
            src/test/java/hudson/security/LDAPEmbeddedTest.java
            http://jenkins-ci.org/commit/ldap-plugin/558839079c074e9e39268d6b8bffbd2932244053
            Log:
            JENKINS-45431 Updated after review

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Francisco Javier Fernandez Gonzalez Path: src/test/java/hudson/security/LDAPEmbeddedTest.java http://jenkins-ci.org/commit/ldap-plugin/558839079c074e9e39268d6b8bffbd2932244053 Log: JENKINS-45431 Updated after review
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Francisco Javier Fernandez Gonzalez
            Path:
            src/test/java/hudson/security/LDAPEmbeddedTest.java
            src/test/java/hudson/security/LDAPSecurityRealmTest.java
            http://jenkins-ci.org/commit/ldap-plugin/b32622d1d0023364c9be8e2c810e1452da30057b
            Log:
            JENKINS-45431 Updated after review

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Francisco Javier Fernandez Gonzalez Path: src/test/java/hudson/security/LDAPEmbeddedTest.java src/test/java/hudson/security/LDAPSecurityRealmTest.java http://jenkins-ci.org/commit/ldap-plugin/b32622d1d0023364c9be8e2c810e1452da30057b Log: JENKINS-45431 Updated after review

              People

              • Assignee:
                rsandell rsandell
                Reporter:
                neej Michael Sjölund
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: