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.

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

Node Mautic unusable for those who using Basic Auth #3760

Closed
luizeof opened this issue Jul 22, 2022 · 6 comments · Fixed by #3761
Closed

Node Mautic unusable for those who using Basic Auth #3760

luizeof opened this issue Jul 22, 2022 · 6 comments · Fixed by #3761

Comments

@luizeof
Copy link
Contributor

luizeof commented Jul 22, 2022

Describe the bug

N8N version 0.187.1 modifies node Mautic to add Credentials Test. #3668

But the /api/users/self endpoint works by OAuth2, for all users using Mautic Basic Auth API the node Mautic is broken

erro-api-mautic-n8n

To Reproduce
Steps to reproduce the behavior:

  1. Upgrade N8N
  2. Authenticate to Mautic Node using OAuth2 (it works)
  3. Authenticate to Mautic Node using Basic Auth (not work)

Expected behavior

Mautic allows accessing the API by two modes: Basic Auth and OAuth, allowing N8N to work with both modes.

Environment (please complete the following information):

  • OS: Debian Docker
  • n8n Version: 0.187.2
  • Node.js Version [e.g. 14.16.0]
  • Database system MySQL 8
  • Operation mode: Queue

Additional context
Add any other context about the problem here.

@Joffcom
Copy link
Member

Joffcom commented Jul 22, 2022

Hey @luizeof,

Thanks for reporting this one, I am just testing a fix now.

@Joffcom Joffcom linked a pull request Jul 22, 2022 that will close this issue
@janober
Copy link
Member

janober commented Jul 22, 2022

But does the node not still work perfectly? The test should not matter at all for the functionality, is just a frontend thing .

@luizeof
Copy link
Contributor Author

luizeof commented Jul 22, 2022

@janober not work at all (only using Basic Auth)

Using OAuth works perfectly as usual.

@janober
Copy link
Member

janober commented Jul 22, 2022

Ah then more than just the test got updated. We will have a look.

@janober janober added the Upcoming Release Will be part of the upcoming release label Jul 24, 2022
@janober
Copy link
Member

janober commented Jul 24, 2022

Fix got merged and will be released with the next version.

@janober
Copy link
Member

janober commented Jul 27, 2022

Fix got released with n8n@0.188.0

@janober janober removed the Upcoming Release Will be part of the upcoming release label Jul 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants