Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

no function clause matching in :lists.usort/2 #280

Closed
maennchen opened this issue Oct 18, 2023 Discussed in #279 · 1 comment · Fixed by #281
Closed

no function clause matching in :lists.usort/2 #280

maennchen opened this issue Oct 18, 2023 Discussed in #279 · 1 comment · Fixed by #281
Assignees
Labels
Milestone

Comments

@maennchen
Copy link
Member

Discussed in #279

Originally posted by DnOberon October 17, 2023
Hello it's me again!

Trying to create redirect URL gives me this error - using the config gleaned from this (if you can reach it https://identity-preview.inl.gov/.well-known/openid-configuration)

[error] GenServer #PID<0.528.0> terminating
** (FunctionClauseError) no function clause matching in :lists.usort/2
    (stdlib 5.0.2) lists.erl:1259: :lists.usort(#Function<1.25876173/2 in :oidcc_authorization.deprioritize_none_alg/1>, :undefined)
    (oidcc 3.0.1) src/oidcc_authorization.erl:233: :oidcc_authorization.attempt_request_object/2
    (oidcc 3.0.1) src/oidcc_authorization.erl:83: :oidcc_authorization.create_redirect_url/2

I've verified my context is being pulled in correctly, and can give the  dbg output of that on request. 
    {:ok, context} =
      Oidcc.ClientContext.from_configuration_worker(
        Ingest.Application.OneID,
        config[:client_id],
        config[:client_secret]
      )

    dbg(
      Oidcc.Authorization.create_redirect_url(context, %{
        redirect_uri: "http://localhost:4000/users/log_in"
      })
    )

On the latest version.

@maennchen maennchen added the bug label Oct 18, 2023
@maennchen
Copy link
Member Author

This is an internal bug. Values are read from the metadata. It crashes if they’re not defined even though they’re optional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant