coolify icon indicating copy to clipboard operation
coolify copied to clipboard

[Bug]: Penpot SERVICE_FQDN_FRONTEND environmental variable has a trailing slash

Open HarleySalas opened this issue 1 year ago • 2 comments

Description

With the Penpot service install, it adds a trailing slash to the SERVICE_FQDN_FRONTEND and technically, that is a url, not FQDN.I see this pattern a lot with services; the variables are stated as an "FQDN," but an FQDN is simply a host and address, like "example.com" or "sub.example.com," not the https:// prefix, nor a trailing /.

Minimal Reproduction (if possible, example repository)

Just do a one click install of Penpot

Exception or Error

The issue is that for the emails penpot sends, it uses the FQDN to construct it's callback urls and it adds it's own trailing slash, as it expects a FQDN. The result is that we end up with not found pages, instead of getting the correct callback. image

This is the sort of url that comes out, note the two forward slashes after .com// https://penpot.mydomain.com//#/auth/recovery?token=(redacted)

Version

297

HarleySalas avatar Jun 22 '24 13:06 HarleySalas

https://twenty.mywebsite.com//graphql also broke my twenty crm, which is live and being used

HarleySalas avatar Jun 22 '24 14:06 HarleySalas

yes needs to be fixed asap

GunniBusch avatar Jun 22 '24 22:06 GunniBusch

Just noticed this today SERVICE_URL_APPWRITE is expected to generate something like appwrite-vgsco4o.example.com instead it generates with trailing slash appwrite-vgsco4o.example.com/

carbontwelve avatar Jul 12 '24 22:07 carbontwelve

👀

nicanordlc avatar Jul 31 '24 03:07 nicanordlc

Yes, FQDN and URL are mixed up in some places (I will fix this in the near future). The trailing slash problem was fixed in one of the latest updates, so this is no longer an issue.

peaklabs-dev avatar Oct 28 '24 13:10 peaklabs-dev