Cloudron makes it easy to run web apps like WordPress, Nextcloud, GitLab on your server. Find out more or install now.


    Cloudron Forum

    • Register
    • Login
    • Search
    • Categories
    • Recent
    • Tags
    • Popular

    Multiple proxyAuth Path's

    N8N
    3
    7
    356
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • T
      thetomester13 App Dev last edited by

      I know this was discussed previously during the implementation of the proxyAuth manifest attribute, but is there a way to disable proxyAuth for multiple paths? For example, in N8N we blocklist /webhook to expose this to the WWW. However, I'm now seeing that there's also a /rest that would be useful to expose for creating Oauth credentials. See here:

      0a0b2aaf-bf36-415a-8cd1-3866380daac6-image.png

      1 Reply Last reply Reply Quote 1
      • ?
        A Former User last edited by

        Odd... the guide seems to imply that you can but its not explicit about it. @staff is this true?

        1 Reply Last reply Reply Quote 0
        • ?
          A Former User last edited by

          Just read the docs again and IDK how I got the impression we can do that. Seems like its just one. Ideally we could do something like:

          "proxyAuth": { "paths": [
            "!/webhook",
            "!/rest",
          ] },
          
          1 Reply Last reply Reply Quote 2
          • girish
            girish Staff last edited by

            Right, only one path was implemented since I wanted a use case for having multiple ones before implementing it. Looks like I got a use case now...

            T 1 Reply Last reply Reply Quote 4
            • T
              thetomester13 App Dev @girish last edited by

              @girish not to put too much more on your already full plate, but did this request make it onto the roadmap?

              girish 1 Reply Last reply Reply Quote 2
              • girish
                girish Staff @thetomester13 last edited by

                @thetomester13 yeah, i don't think it will make it to this release. Will look into it for the next one.

                T 1 Reply Last reply Reply Quote 1
                • T
                  thetomester13 App Dev @girish last edited by

                  @girish perfect, thanks! I think this could go far in the way of hardening this unstable app.

                  1 Reply Last reply Reply Quote 2
                  • First post
                    Last post
                  Powered by NodeBB