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


  • Categories
  • Recent
  • Tags
  • Popular
  • Bookmarks
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse

Cloudron Forum

Apps | Demo | Docs | Install

Multiple proxyAuth Path's

Scheduled Pinned Locked Moved N8N
7 Posts 3 Posters 360 Views
    • 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 Offline
    T Offline
    thetomester13 App Dev
    wrote on last edited by
    #1

    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
    1
  • ? Offline
    ? Offline
    A Former User
    wrote on last edited by
    #2

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

    1 Reply Last reply
    0
  • ? Offline
    ? Offline
    A Former User
    wrote on last edited by
    #3

    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
    2
  • girishG Offline
    girishG Offline
    girish Staff
    wrote on last edited by
    #4

    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
    4
  • T Offline
    T Offline
    thetomester13 App Dev
    replied to girish on last edited by
    #5

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

    girishG 1 Reply Last reply
    2
  • girishG Offline
    girishG Offline
    girish Staff
    replied to thetomester13 on last edited by
    #6

    @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
    1
  • T Offline
    T Offline
    thetomester13 App Dev
    replied to girish on last edited by
    #7

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

    1 Reply Last reply
    2

  • Login

  • Don't have an account? Register

  • Login or register to search.
  • First post
    Last post
0
  • Categories
  • Recent
  • Tags
  • Popular
  • Bookmarks
  • Login

  • Don't have an account? Register

  • Login or register to search.