photoprism / photoprism

AI-Powered Photos App for the Decentralized Web πŸŒˆπŸ’Žβœ¨
https://www.photoprism.app
Other
34.16k stars 1.88k forks source link

Use PHOTOPRISM_SITE_URL to create share links in the UI #752

Closed elapse2039 closed 3 years ago

elapse2039 commented 3 years ago

When creating an Album sharing link, the URL includes the local IP address, not the value in PHOTOPRISM_SITE_URL. This was discovered when attempting to place photoprism behind a nginx proxy (for SSL) and using the internal network address (due to DNS/routing issues).

Expecting that PHOTOPRISM_SITE_URL would be used to form the sharing link. Instead, the link includes the local network URL.

Docker image: 201227-91acaaa5-Linux-x86_64

lastzero commented 3 years ago

Are you using this site url to access the server? The URL is generated via JS based on the host you are currently using.

lastzero commented 3 years ago

We've implemented it this way (for now) as we expect many users to enter a wrong URL into PHOTOPRISM_SITE_URL or ignore this config option completely when "just" testing.

elapse2039 commented 3 years ago

From within my local network where the photo prism server is running I do not use the public URL because of routing issues to the NGINX proxy. From outside, it works as expected.

However, it sounds like from your description that PHOTOPRISM_SITE_URL is not even used? This kind of parameter is typical/expected in many other opensource web-based projects I've installed.

Anyway, I can't use photo prism to share albums etc unless I'm outside of my local network. :(

lastzero commented 3 years ago

We can improve this. As mentioned, there may be other users who ignore this config option and are disappointed because it doesn't use the current hostname. We do our very best to make everyone happy!

lastzero commented 3 years ago

Note that you can manually change the URL - the important parts are the share slug & token after the host name.

elapse2039 commented 3 years ago

Ok, that may be a good workaround for now. Maybe you can compromise my leaving the PHOTOPRISM_SITE_URL as empty string and using JS host and if someone like me fills the PHOTOPRISM_SITE_URL with a string then use that?

elapse2039 commented 3 years ago

It's also not very obvious that this setting in the docker-compose file is not used... maybe it needs removing/comment in the interim?

lastzero commented 3 years ago

Commenting it is the same amount of work than changing it. You're simply the first user who raises this issue and we had more than enough other tasks to take care of. It's Sunday night... companies like Adobe won't even reply to your questions at this time :)

lastzero commented 3 years ago

Next build contains this improvement for you, so that the configured site url is used unless it happens to be empty :+1:

site_url

lastzero commented 3 years ago

Docker build has been started: https://drone.photoprism.app/photoprism/photoprism/600

Please use photoprism/photoprism:preview for testing preview builds.

lastzero commented 3 years ago

@elapse2039 That's what I was talking about: #941

7IM3rW commented 3 years ago

Hi! I am not a developer so I hope this is the right place to ask. Is it planned that the receipient of the link will have to enter a password? So I could share pictures with my family and friends, but its more secure if a password is needed in case some stranger accidentaly "finds" the link. Thanks!

lastzero commented 3 years ago

Yes, it is planned πŸ‘

xhabih commented 2 years ago

I've set the PHOTOPRISM_SITE_URL but I still have "localhost" when sharing, I this not yet implemented?

lastzero commented 2 years ago

Did you restart for changes to take effect? Is it a valid URL?

xhabih commented 2 years ago

Restarted and now working OK. Thanks.

‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ El jueves, 11 de noviembre de 2021 a las 13:18, Michael Mayer @.***> escribiΓ³:

Did you restart for changes to take effect? Is it a valid URL?

β€” You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

ant0nwax commented 2 years ago

Hi

I changed the PHOTOPRISM_SITE_URL to "https://prism.example.com/" I restarted with docker-compose stop and docker-compose start and still see localhost:2324, is there a way how to restart correctly? maybe i am not totally aware that restart could be something different too? Thank you

lastzero commented 2 years ago

Use up, not start: https://docs.photoprism.app/getting-started/docker-compose/#step-2-start-the-server

Please note that GitHub Issues should not be used as support forum :)

ant0nwax commented 2 years ago

@lastzero you are a hero for a docker pebkac

kmanan commented 1 year ago

I seem to be running into a variation of this issue. I am accessing photoprism my domain xyz.com but the URL generated for sharing an album is photoprism.me:2342 and this URL does not work. I have installed Photoprism in TrueNAS and I don't know which file I would need to edit to get the right domain name configured.

Bonscha commented 11 months ago

Hi, I have another issue altogether. I might be setting the variable in the wrong way or it might be used in a way I don't think of, but...

I connect via my own DynDNS to the server, which looks like this:

http://dyndns.MYDOMAIN.TLD/index.php?protocol=http&port=MYPORT

I can add a path to this URL, which does result in a shared link open correctly: http://dyndns.MYDOMAIN.TLD/index.php?protocol=http&port=MYPORT&pfad=/x/73nhz72t6u3htu2/GALLERY

When I add the environment variable in docker (CasaOS) though, it doesn't even show the standard UI after restarting the container. I can't even use Photoprism at all.

What I entered is the following: PHOTOPRISM_SITE_URL = http://dyndns.MYDOMAIN.TLD/index.php?protocol=http&port=MYPORT&pfad=

Can someone explain to my why the UI doesn't show up anymore, when this variable is set, and how I can use my own DynDNS path for sharing link generation?

Sebastian