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

Cie Integration #311

Merged
merged 24 commits into from
Mar 13, 2024
Merged

Cie Integration #311

merged 24 commits into from
Mar 13, 2024

Conversation

rglauco
Copy link
Collaborator

@rglauco rglauco commented Mar 11, 2024

Added distinction between key usage during RP and OP operations
Added example encryption keys in TA->RP and RP db dumps
Added encryption algs to metadata validator
Fixed entity_type save method

@rglauco rglauco requested a review from peppelinux March 11, 2024 16:51
Copy link
Member

@peppelinux peppelinux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

avanziamo di versione in __init__.py come consueto

spid_cie_oidc/entity/models.py Outdated Show resolved Hide resolved
spid_cie_oidc/provider/views/userinfo_endpoint.py Outdated Show resolved Hide resolved
Copy link
Collaborator Author

@rglauco rglauco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all done

Copy link
Member

@peppelinux peppelinux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

facciamo una minor version perché non è un bugfix ma una feature aggiuntiva

@@ -263,7 +263,6 @@ def set_jwks_as_array(self):
setattr(self, i, [value])

def save(self, *args, **kwargs):
self.entity_type = self.type[0]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perché è rimosso? prende il primo entity type disponibile come quello default

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perché quando si modifica l'EC di un RP nel db viene salvato il valore federation_entity a prescindere dal valore della select, che sta in self.type[1] e non viene preso

spid_cie_oidc/entity/utils.py Outdated Show resolved Hide resolved
spid_cie_oidc/entity/utils.py Outdated Show resolved Hide resolved
@peppelinux peppelinux merged commit 54f8e88 into italia:main Mar 13, 2024
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants