Code development platform for open source projects from the European Union institutions

Skip to content
Snippets Groups Projects

Add new SAML Configuration for a Mapping

Merged Vincent SIMONIN requested to merge saml_config into main
1 unresolved thread

Mapping can now have SAML Coniguration:

  • Only API was implemented

Merge request reports

Pipeline #77619 passed

Pipeline passed for f6932441 on saml_config

Approval is optional

Merged by Laurent VENIERLaurent VENIER 1 year ago (Jul 28, 2023 8:15am UTC)

Merge details

  • Changes merged into main with 91debd9f.
  • Deleted the source branch.

Pipeline #77628 passed

Pipeline passed for 91debd9f on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
72 ordering = ("source", "target")
69 73
70 74 def __str__(self):
71 return f'{self.source}'
75 return f"{self.source}"
72 76
73 77 def get_absolute_url(self):
74 return reverse('plugins:netbox_rps_plugin:mapping', args=[self.pk])
78 return reverse("plugins:netbox_rps_plugin:mapping", args=[self.pk])
79
75 80
76 def get_authentication_color(self):
77 return AuthenticationChoices.colors.get(self.authentication)
81 class SamlConfig(NetBoxModel):
82 acs_url = models.CharField(
83 max_length=2000,
  • I would add 4 tests:

    • a simple PATCH
    • url regex on acs and logout
    • test limit 2000
    • a single POST to create mapping+header+saml
  • Vincent SIMONIN added 1 commit

    added 1 commit

    • f6932441 - Add new SAML Configuration for a Mapping

    Compare with previous version

  • Laurent VENIER approved this merge request

    approved this merge request

  • Laurent VENIER mentioned in commit 91debd9f

    mentioned in commit 91debd9f

  • merged

  • Please register or sign in to reply
    Loading