-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Milestone
Description
Blocked due to client support as in inrupt/solid-client-authn-js#3181
Also must prune following temporary adjustments, that were introduced for client support.
ath
claim in dpop proofs must be mandatory. and following temp-fix must be resolved.manas/fcrates/dpop/src/proof/validated.rs
Lines 118 to 125 in bef113e
let decoded_ath = decoded_claims .ath .as_ref() // TODO MUST remove following feature and block. .or_else(|| cfg!(feature = "unsafe-optional-ath-claim").then_some(&ath)) .ok_or(InvalidDPoPProof::AthClaimMismatch)?; azp
claimis currenly (de)serialized asclient_id
for ecosystem interoperability.manas/fcrates/solid_oidc_types/src/id_token/payload.rs
Lines 23 to 28 in bef113e
/// Authorized party. #[serde(alias = "client_id")] pub azp: String, /// Audience. aud
claim currently accepts single value too.manas/fcrates/solid_oidc_types/src/id_token/payload.rs
Lines 27 to 33 in bef113e
/// Audience. // NOTE: for NSS idp compat, allows deserializing from a string. // TODO remove special handling. #[serde(deserialize_with = "string_or_vec")] pub aud: Vec<String>,
Once solid-client follows the spec, and solidos follows, these changes will be merged.