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

[Bug]: Basic auth with traefik labels stopped working (404 page not found) #2202

Closed
RayBB opened this issue May 14, 2024 · 13 comments · Fixed by #2205
Closed

[Bug]: Basic auth with traefik labels stopped working (404 page not found) #2202

RayBB opened this issue May 14, 2024 · 13 comments · Fixed by #2205
Assignees

Comments

@RayBB
Copy link
Contributor

RayBB commented May 14, 2024

Description

Today I tried to go to an application running on coolify and got the message 404 page not found
I tried a few more and noticed all the ones behind basic auth are doing that and the rest are fine.
If I remove the label from one of them the app is accessible.
If I put it back on it 404s right away.

Example label:

    labels:
      - 'traefik.http.middlewares.custom-auth.basicauth.users=test:$$2y$$12$$ci.4U63YX83CwkyUrjqxAucnmi2xXOIlEF6T/KdP9824f1Rf1iyNG'

Is anyone else experiencing this? I haven't touched anything in coolify between using the app yesterday and everything working fine and today getting the 404?

Minimal Reproduction (if possible, example repository)

I don't know how to reproduce this.

Exception or Error

No response

Version

v4.0.0-beta.277

@charlesilieff
Copy link

I think I have a linked issue, traefik dashboard works only if I disable the basic auth (removing : ```
middlewares:
auth:
basicAuth:
users:
- "<GENERATED_USERNAME>:<GENERATED_PASSWORD>"

@RayBB
Copy link
Contributor Author

RayBB commented May 14, 2024

@charlesilieff what is the url to the traefik dashboard?

@charlesilieff
Copy link

charlesilieff commented May 14, 2024

http://traefik.my-domain.com/dashboard/

@andrasbacsai
Copy link
Member

Did you try with simple $ as described in the docs?

https://coolify.io/docs/knowledge-base/traefik/basic-auth

@RayBB
Copy link
Contributor Author

RayBB commented May 14, 2024

@andrasbacsai here is a recording of the issue. I copied the strings straight from the docs.
So weird because it has been working for weeks and changed all of a sudden.

here is a video:

coolifybug2.mp4

@andrasbacsai andrasbacsai self-assigned this May 15, 2024
Copy link
Member

Found the issue. Fixing in the next version.

@RayBB
Copy link
Contributor Author

RayBB commented May 15, 2024

@andrasbacsai
thanks! btw, this is probably such a commonly needed feature it might make sense to make a little GUI for it.

@fernandops26
Copy link

@RayBB @andrasbacsai i just had the same issue, what is the workaround?

@RayBB
Copy link
Contributor Author

RayBB commented May 15, 2024

@fernandops26 no workaround afaik. but beta.278 should probably be out tomorrow or so with a fix.

@RayBB
Copy link
Contributor Author

RayBB commented May 16, 2024

@andrasbacsai I am on v4.0.0-beta.279 and still experiencing this issue.

I have tried:

  • With the checkbox checked and not.
  • And with quotes and not.
  • Redeploying each time between.

It 404s in all cases except when I remove the label.

Is there something else I am missing?

@fernandops26 did the new version fix it for you?

@frans2526
Copy link

frans2526 commented May 17, 2024

Hello,

I am also experiencing the same issue on version v4.0.0-beta.280. I followed the same steps as RayBB, but I still encountered a 404 error.

Thank you.

@andrasbacsai andrasbacsai reopened this May 17, 2024
@RayBB
Copy link
Contributor Author

RayBB commented May 27, 2024

I have no idea what changed but on beta 286 it is working just like this:
image

@RayBB
Copy link
Contributor Author

RayBB commented May 27, 2024

Created a PR to improve the documentation coollabsio/documentation-coolify#15

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.

5 participants