[keycloak-dev] KEYCLOAK-5032 - Implementation question

carl-kristian.eriksen at telia.no carl-kristian.eriksen at telia.no
Mon Oct 2 07:34:05 EDT 2017


The second commit in the PR failed the CI build, but the tests does not fail locally.
Do you have any suggestions on how to handle this?

Carl Kristian Eriksen


On 29/09/17 16:41, "keycloak-dev-bounces at lists.jboss.org on behalf of carl-kristian.eriksen at telia.no" <keycloak-dev-bounces at lists.jboss.org on behalf of carl-kristian.eriksen at telia.no> wrote:

    Hi.
    
     I’ll follow up on the PR.
    
    Br / mvh
    Carl Kristian Eriksen
    On 29/09/17 15:50, "Bill Burke" <bburke at redhat.com> wrote:
    
        Do you want to continue talking on the PR or here?  I had some
        concerns with your PR.
        
        On Wed, Sep 27, 2017 at 5:45 AM,  <carl-kristian.eriksen at telia.no> wrote:
        > https://issues.jboss.org/browse/KEYCLOAK-5032 describes two requested query parameters: acr_values and nonce
        >
        > Our requirements are for acr_values and prompt, and I’m working on a pull request for these two.
        >
        > How many pull requests do you want?
        > Should I make sure that (each)PR includes support for one, two or three query parameters
        >
        > Can the “prompt” parameter be added to KEYCLOAK-5032, or do I need another Jira task for the “prompt” parameter?
        >
        > Br / mvh
        > Carl Kristian Eriksen
    > _______________________________________________
        > keycloak-dev mailing list
        > keycloak-dev at lists.jboss.org
        > https://lists.jboss.org/mailman/listinfo/keycloak-dev
        
        
        
        -- 
        Bill Burke
        Red Hat
        
    
    
    _______________________________________________
    keycloak-dev mailing list
    keycloak-dev at lists.jboss.org
    https://lists.jboss.org/mailman/listinfo/keycloak-dev




More information about the keycloak-dev mailing list