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

[openid-plugin] Support 3 different email attributes for AX Extension

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      The standard email attribute type URI for the AX extension is "http://axschema.org/contact/email". However, many of the OpenId providers do not support this standard attribute type URI but instead support one of the following:

      The Jenkins openid-plugin currently supports only "http://schema.openid.net/contact/email". For a better user experience, it is recommended to ask/support for all three email attributes.

        Attachments

          Activity

          Hide
          jssnirmal Nirmal Jonnalagedda added a comment -
          Show
          jssnirmal Nirmal Jonnalagedda added a comment - Link to the pull request made: https://github.com/jenkinsci/openid-plugin/pull/6
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Nirmal Jonnalagedda
          Path:
          src/main/java/hudson/plugins/openid/impl/UserInfoExtension.java
          src/test/java/hudson/plugins/openid/OpenIdAXEmailAttributesTest.java
          src/test/java/hudson/plugins/openid/OpenIdTestCase.java
          src/test/java/hudson/plugins/openid/OpenIdTestService.java
          http://jenkins-ci.org/commit/openid-plugin/d5eeed596d5791017bca422286f9de944cdcd8e6
          Log:
          [FIXED JENKINS-23009] Try 3 different email AX attributes

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Nirmal Jonnalagedda Path: src/main/java/hudson/plugins/openid/impl/UserInfoExtension.java src/test/java/hudson/plugins/openid/OpenIdAXEmailAttributesTest.java src/test/java/hudson/plugins/openid/OpenIdTestCase.java src/test/java/hudson/plugins/openid/OpenIdTestService.java http://jenkins-ci.org/commit/openid-plugin/d5eeed596d5791017bca422286f9de944cdcd8e6 Log: [FIXED JENKINS-23009] Try 3 different email AX attributes
          Hide
          jssnirmal Nirmal Jonnalagedda added a comment -

          There is a merge conflict for the pull request #6. Please look into it.

          Thanks

          Show
          jssnirmal Nirmal Jonnalagedda added a comment - There is a merge conflict for the pull request #6 . Please look into it. Thanks

            People

            Assignee:
            kohsuke Kohsuke Kawaguchi
            Reporter:
            jssnirmal Nirmal Jonnalagedda
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: