Skip to content

Commit

Permalink
[FIX] auth_saml_environment: no require on readonly fields
Browse files Browse the repository at this point in the history
Restore the usage of the form view by dropping the required condition on
binary fields that are readonly.

Those fields became mandatory after OCA/server-auth#342
but the module auth_saml_environment offer to use paths as a alternative config
option for binaries thus those fields can be empty.
  • Loading branch information
StephaneMangin authored and yvaucher committed Mar 18, 2024
1 parent 2ba0fa5 commit dff21d7
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions auth_saml_environment/models/auth_saml_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,11 @@ class AuthSamlProvider(models.Model):
_name = "auth.saml.provider"
_inherit = ["auth.saml.provider", "server.env.mixin"]

# Non-mandatory to be able to create objects
idp_metadata = fields.Text(required=False)
sp_pem_public = fields.Char(required=False)
sp_pem_private = fields.Char(required=False)

sp_pem_public_path = fields.Char(
string="sp_pem_public_path env config value",
)
Expand Down

0 comments on commit dff21d7

Please sign in to comment.