immich-app / immich

High performance self-hosted photo and video management solution.
https://immich.app
GNU Affero General Public License v3.0
51.74k stars 2.74k forks source link

Changing the photo’s Location on the web page has no effect. #14006

Open sunxiang0918 opened 5 days ago

sunxiang0918 commented 5 days ago

The bug

I’m attempting to change the photo’s geographical location on a web page. However, I’ve noticed that modifying the location only works if I select the position on the map. Simply entering the latitude and longitude manually on the interface and clicking ‘Confirm’ doesn’t actually change the location. It seems that no request is made to the server (PUT /api/assets) in this case.

image

The OS that Immich Server is running on

Docker

Version of Immich Server

v1.120.1

Version of Immich Mobile App

v1.120.1

Platform with the issue

Your docker-compose.yml content

name: immich

services:

  immich-server:
    container_name: immich_server
    image: ghcr.io/immich-app/immich-server:${IMMICH_VERSION:-release}
    privileged: true
    devices:
      - /dev/dri:/dev/dri
    ports:
      - 3001:2283
    volumes:
      # Do not edit the next line. If you want to change the media storage location on your system, edit the value of UPLOAD_LOCATION in the .env file
      - ${UPLOAD_LOCATION}:/usr/src/app/upload
      - /etc/localtime:/etc/localtime:ro
      - /volume2/photo/PhotoLibrary:/synologyPhotos
      # geo
      - /volume1/docker/immich/geo/geodata:/build/geodata 
      - /volume1/docker/immich/geo/i18n-iso-countries:/usr/src/app/node_modules/i18n-iso-countries
    env_file:
      - .env
    depends_on:
      - redis
      - database
    restart: always
    healthcheck:
      disable: false
    networks:
      - nas_network

  immich-machine-learning:
    container_name: immich_machine_learning
    # For hardware acceleration, add one of -[armnn, cuda, openvino] to the image tag.
    # Example tag: ${IMMICH_VERSION:-release}-cuda
    image: ghcr.io/immich-app/immich-machine-learning:${IMMICH_VERSION:-release}-openvino
    extends: # uncomment this section for hardware acceleration - see https://immich.app/docs/features/ml-hardware-acceleration
      file: hwaccel.ml.yml
      service: openvino # set to one of [armnn, cuda, openvino, openvino-wsl] for accelerated inference - use the `-wsl` version for WSL2 where applicable
    privileged: true
    volumes:
      - /volume1/docker/immich/model-cache:/cache
    env_file:
      - .env
    restart: always
    healthcheck:
      disable: false
    networks:
      - nas_network

  redis:
    container_name: immich_redis
    image: redis:6.2-alpine@sha256:2d1463258f2764328496376f5d965f20c6a67f66ea2b06dc42af351f75248792
    healthcheck:
      test: redis-cli ping || exit 1
    restart: always
    networks:
      - nas_network

  database:
    container_name: immich_postgres
    image: tensorchord/pgvecto-rs:pg14-v0.2.0@sha256:90724186f0a3517cf6914295b5ab410db9ce23190a2d9d0b9dd6463e3fa298f0
    environment:
      POSTGRES_PASSWORD: ${DB_PASSWORD}
      POSTGRES_USER: ${DB_USERNAME}
      POSTGRES_DB: ${DB_DATABASE_NAME}
      POSTGRES_INITDB_ARGS: '--data-checksums'
    volumes:
      # Do not edit the next line. If you want to change the database storage location on your system, edit the value of DB_DATA_LOCATION in the .env file
      - ${DB_DATA_LOCATION}:/var/lib/postgresql/data
    healthcheck:
      test: pg_isready --dbname='${DB_DATABASE_NAME}' --username='${DB_USERNAME}' || exit 1; Chksum="$$(psql --dbname='${DB_DATABASE_NAME}' --username='${DB_USERNAME}' --tuples-only --no-align --command='SELECT COALESCE(SUM(checksum_failures), 0) FROM pg_stat_database')"; echo "checksum failure count is $$Chksum"; [ "$$Chksum" = '0' ] || exit 1
      interval: 5m
      start_period: 5m
    command: ["postgres", "-c", "shared_preload_libraries=vectors.so", "-c", 'search_path="$$user", public, vectors', "-c", "logging_collector=on", "-c", "max_wal_size=2GB", "-c", "shared_buffers=512MB", "-c", "wal_compression=on"]
    restart: always
    networks:
      - nas_network

networks:
  nas_network:
    external: true

Your .env content

# You can find documentation for all the supported env variables at https://immich.app/docs/install/environment-variables

# The location where your uploaded files are stored
UPLOAD_LOCATION=/volume1/docker/immich/library
# The location where your database files are stored
DB_DATA_LOCATION=/volume1/docker/immich/postgres

# To set a timezone, uncomment the next line and change Etc/UTC to a TZ identifier from this list: https://en.wikipedia.org/wiki/List_of_tz_database_time_zones#List
# TZ=Etc/UTC

# The Immich version to use. You can pin this to a specific version like "v1.71.0"
IMMICH_VERSION=v1.120.1

# Connection secret for postgres. You should change it to a random password
# Please use only the characters `A-Za-z0-9`, without special characters or spaces
DB_PASSWORD=xxxxxxxxxx

# The values below this line do not need to be changed
###################################################################################
DB_USERNAME=xxxxxxxxxx
DB_DATABASE_NAME=immich

Reproduction steps

  1. Choose one or more photos.
  2. Tap ‘Change Location’.
    1. Input the latitude and longitude directly without selecting a point on the map.
    1. Press ‘Confirm’. ...

Relevant log output

No response

Additional information

No response

bo0tzz commented 5 days ago

I can't reproduce this. Are you using a reverse proxy or anything like that? Is the asset in an external library?

sunxiang0918 commented 2 days ago

I replicated the procedure and conducted several tests, which indicated that the issue is unrelated to reverse proxy settings or extension libraries.

Entering the latitude and longitude directly into the input field using the keyboard works fine.

The problem arises when a specific latitude and longitude, such as ‘30.724233,103.81877’, is pasted into the input box all at once. However, if the latitude and longitude are pasted separately—first the latitude, then the longitude—this issue does not occur.

1

alextran1502 commented 2 days ago

@sunxiang0918 Thank for the screen capture, this will help with fixing the bug