-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
website/integrations: add Mautic #13324
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for authentik-storybook canceled.
|
✅ Deploy Preview for authentik-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thank you for opening this PR. Here are a few comments to get this review started. I'll come back with a more in-dept review once some little internal details are sorted out. Please do let me know if you have any questions 🚀
|
||
## What is Mautic | ||
|
||
> The World's Largest Open Source Marketing Automation Product |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you give a better description of the product? Preferably a full/descriptive sentence
This documentation lists only the settings that you need to change from their default values. Be aware that any changes other than those explicitly mentioned in this guide could cause issues accessing your application. | ||
::: | ||
|
||
### Create a self-signed certificate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Under System > Certificates of the admin interface, it's possible for authentik to generate the certificate for you. Is there a reason why generating the certificate manually is preferred? (To download cert/priv key, click the dropdown)
Mautic only supports RSA keys | ||
::: | ||
|
||
Also, Mautic requires the key to mention RSA in the header and footer. Change the private_key.pem manually if necessary: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if necessary
What do you mean by this? The output won't change from one run to the other...
If you implement the previous selection, make sure to check whether the files use the PKCS#8 (-----BEGIN PRIVATE KEY-----
) or PKCS#1 (-----BEGIN RSA PRIVATE KEY-----
) format
``` | ||
This indicates a problem with the mapping of the attributes ([Mautic configuration](#mautic-configuration) > Step 2 > Email/Username/First Name/Last Name or [authentik configuration](#authentik-configuration) > Step 2). | ||
- _"Unable to verify Signature"_ when logging in: The certificate does not match the key. (E.g. when the certificate was generated without the `RSA`.) | ||
- _"Assertions must be signed"_ when logging in: Are the checkboxes mentioned in [authentik configuration](#authentik-configuration) > Step 3 checked? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a "Configuration verification" section? https://github.com/goauthentik/authentik/blob/main/website/integrations/template/service.md?plain=1#L40
``` | ||
3. Create an [SAML provider](../../../docs/add-secure-apps/providers/saml) with the following values: | ||
- **ACS URL**: <kbd>https://<em>mautic.company</em>/s/saml/login_check</kbd> | ||
- **Issuer**: <kbd><em>mautic.company</em></kbd> (without `https://`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since you specify the value to copy in a codeblock, you don't need to tell the user to not include the protocol
Details
Add Mautic integration to Chat, Communication & Collaboration.
Not applicable (I hope)
## Checklistak test authentik/
)make lint-fix
)If an API change has been made
make gen-build
)If changes to the frontend have been made
make web
)Applicable
make website
)