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.

Sign up for GitHub

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

Switched from metadata_url to metadata for end user authentication #26042

Open
wants to merge 24 commits into
base: main
Choose a base branch
from
+6 −2
Open

Switched from metadata_url to metadata for end user authentication #26042

Changes from 17 commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
1d78907
Update default.yml
allenhouchins Feb 4, 2025
e43bb7f
Update default.yml
allenhouchins Feb 4, 2025
b560a28
Update default.yml
allenhouchins Feb 4, 2025
07590f5
Update default.yml
allenhouchins Feb 4, 2025
3beb660
Update default.yml
allenhouchins Feb 4, 2025
bbd2ca4
Update default.yml
allenhouchins Feb 4, 2025
8a96064
Update default.yml
allenhouchins Feb 4, 2025
113a754
Update default.yml
allenhouchins Feb 4, 2025
f8965b0
Update default.yml
allenhouchins Feb 5, 2025
bcc95ee
Update default.yml
allenhouchins Feb 5, 2025
cd9833d
Let's see if this works
noahtalerman Feb 5, 2025
0c9f164
Update default.yml
allenhouchins Feb 5, 2025
97859c1
Fix typo
noahtalerman Feb 5, 2025
48ebdb7
Merge branch 'allenhouchins-end-user-metadata-field' of https://githu…
allenhouchins Feb 5, 2025
a6e9205
Maybe matching the docs will work
noahtalerman Feb 5, 2025
23f4e18
Update default.yml
allenhouchins Feb 5, 2025
f230762
Update default.yml
allenhouchins Feb 6, 2025
2927aa3
Update default.yml
allenhouchins Feb 11, 2025
c1bc859
Update default.yml
allenhouchins Feb 11, 2025
eed01dc
Update default.yml
allenhouchins Feb 11, 2025
2c2cff8
updated variable
allenhouchins Feb 11, 2025
a07d892
Update default.yml
allenhouchins Feb 11, 2025
96a7e39
Update default.yml
allenhouchins Feb 11, 2025
6934cc9
Update default.yml
allenhouchins Feb 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions it-and-security/default.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,10 @@ org_settings:
end_user_authentication:
entity_id: dogfood-eula.fleetdm.com
idp_name: Google Workspace
metadata_url: $DOGFOOD_MDM_SSO_METADATA_URL
issuer_uri: $DOGFOOD_SSO_ISSUER_URI
metadata: |-
$DOGFOOD_SSO_METADATA
Copy link
Contributor

Choose a reason for hiding this comment

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

@allenhouchins based on my testing for #26135 I would experiment with moving this two spaces forward or two spaces back, i.e.

      metadata: |-
      $DOGFOOD_SSO_METADATA

or

      metadata: |-
          $DOGFOOD_SSO_METADATA

and see if either of those works.

Copy link
Member Author

Choose a reason for hiding this comment

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

@sgress454 No change in behavior.
Two spaces forward: 2927aa3
Two spaces backward: c1bc859

Copy link
Contributor

Choose a reason for hiding this comment

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

👍 thanks for trying

metadata_url: ""
apple_business_manager:
- organization_name: Fleet Device Management Inc.
macos_team: "💻 Workstations"
Expand Down Expand Up @@ -56,7 +59,7 @@ org_settings:
enable_sso_idp_login: false
entity_id: dogfood.fleetdm.com
idp_image_url: ""
idp_name: Google
idp_name: Google Workspace
issuer_uri: $DOGFOOD_SSO_ISSUER_URI
metadata: |-
$DOGFOOD_SSO_METADATA
Expand Down
Loading