Skip to content

Commit

Permalink
Fix copy-pasteability of code example
Browse files Browse the repository at this point in the history
Though this code is already not copy-pasteable as valid Ruby (due to the trailing `...` at the end of the `idp_cert_multi` array), that's at least obviously invalid and fast enough to fix.

This missing comma, however, causes parser errors that are less obvious and should be added!
  • Loading branch information
tjschuck authored Jan 8, 2024
1 parent 3463fdd commit 3eb8942
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ use OmniAuth::Strategies::SAML,
:idp_cert_multi => {
:signing => ["-----BEGIN CERTIFICATE-----\n...-----END CERTIFICATE-----", "-----BEGIN CERTIFICATE-----\n...-----END CERTIFICATE-----", ...],
:encryption => []
}
},
:idp_cert_fingerprint => "E7:91:B2:E1:...",
:idp_cert_fingerprint_validator => lambda { |fingerprint| fingerprint },
:name_identifier_format => "urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress"
Expand All @@ -64,7 +64,7 @@ Rails.application.config.middleware.use OmniAuth::Builder do
:idp_cert_multi => {
:signing => ["-----BEGIN CERTIFICATE-----\n...-----END CERTIFICATE-----", "-----BEGIN CERTIFICATE-----\n...-----END CERTIFICATE-----", ...],
:encryption => []
}
},
:idp_cert_fingerprint => "E7:91:B2:E1:...",
:idp_cert_fingerprint_validator => lambda { |fingerprint| fingerprint },
:name_identifier_format => "urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress"
Expand Down

0 comments on commit 3eb8942

Please sign in to comment.