nextcloud / integration_google

🇬 Google integration into Nextcloud
GNU Affero General Public License v3.0
104 stars 33 forks source link

--- In file name when imported from google drive #175

Closed amgschnappi closed 1 month ago

amgschnappi commented 1 month ago

Which version of integration_google are you using?

Latest

Which version of Nextcloud are you using?

Latest

Which browser are you using? In case you are using the phone App, specify the Android or iOS version and device please.

No response

Describe the Bug

  1. Install nc
  2. Install google integration
  3. Give client id and secret
  4. Data migrate

File name has only ---- Not the actual file name

Screenshot 2024-08-07 230144

Expected Behavior

Actual filename from google drive should be shown

To Reproduce

Prereq: Docker compose as below

name: nextcloud
services:
  app:
    image: nextcloud
    restart: always
    ports:
      - 8088:80
    volumes:
      - ..\NextCloud\data:/var/www/html/data
    environment:
      OVERWRITEPROTOCOL: https
      POSTGRES_HOST: db
      POSTGRES_PASSWORD: nextcloud
      POSTGRES_DB: nextcloud
      POSTGRES_USER: nextcloud
  db:
    image: postgres
    environment:
      POSTGRES_PASSWORD: nextcloud
      POSTGRES_DB: nextcloud
      POSTGRES_USER: nextcloud
    restart: always
    volumes:
      - ..\NextCloud\db_data:/var/lib/postgresql/data
    expose:
      - 5432
  1. Install nextcloud on win11 with docker desktop using following batch file :
cd %UserProfile%\Documents\Docker\NextCloud

docker compose stop
docker compose pull
docker compose up -d --remove-orphans

timeout /t 999

 call docker exec --user www-data -it nextcloud-app-1 php occ config:system:get trusted_domains
 call docker exec --user www-data nextcloud-app-1 php /var/www/html/cron.php
 call docker exec --user www-data -it nextcloud-app-1 php occ config:system:set maintenance_window_start --type=integer --value=100
 call docker exec --user www-data -it nextcloud-app-1 php occ db:add-missing-indices
 call docker exec --user www-data -it nextcloud-app-1 php occ app:install richdocumentscode
 call docker exec --user www-data -it nextcloud-app-1 php occ app:install richdocuments
 call docker exec --user www-data -it nextcloud-app-1 php occ app:install memories
 call docker exec --user www-data -it nextcloud-app-1 php occ app:install recognize
 call docker exec --user www-data -it nextcloud-app-1 php occ recognize:download-models
 call docker exec --user www-data -it nextcloud-app-1 php occ app:install integration_google
  1. Create and login with admin
  2. Enter client id and client secret in connected accounts menu of setting
  3. Use data migration menu to import all the google drive files
github-actions[bot] commented 1 month ago

Hello :wave:

Thank you for taking the time to open this issue with integration_google. I know it's frustrating when software causes problems. You have made the right choice to come here and open an issue to make sure your problem gets looked at and if possible solved. We try to answer all issues and if possible fix all bugs here, but it sometimes takes a while until we get to it. Until then, please be patient. If you are strugging to set up the app, have a look at this video: https://www.youtube.com/watch?v=h_teJPg2wEk Note also that GitHub is a place where people meet to make software better together. Nobody here is under any obligation to help you, solve your problems or deliver on any expectations or demands you may have, but if enough people come together we can collaborate to make this software better. For everyone. Thus, if you can, you could also look at other issues to see whether you can help other people with your knowledge and experience. If you have coding experience it would also be awesome if you could step up to dive into the code and try to fix the odd bug yourself. Everyone will be thankful for extra helping hands! One last word: If you feel, at any point, like you need to vent, this is not the place for it; you can go to the forum, to twitter or somewhere else. But this is a technical issue tracker, so please make sure to focus on the tech and keep your opinions to yourself. (Also see our Code of Conduct. Really.)

I look forward to working with you on this issue Cheers :blue_heart:

marcelklehr commented 1 month ago

Hello @amgschnappi Can you post an original file name?

amgschnappi commented 1 month ago

For example "My resume.pdf" "Scanned driving license.pdf" "Inflation vs investment.xlsx" "Some friends resume.docx" etc

marcelklehr commented 1 month ago

Mmh, so no non-ASCII characters involved, seemingly.

amgschnappi commented 1 month ago

Is it because I have installed the app via batch and not via gui?

marcelklehr commented 1 month ago

I can confirm the observation, this is a bug. Thank you for reporting this!

marcelklehr commented 1 month ago

We recently added a blacklist for non-word characters, but a mistake was made, it seems.

marcelklehr commented 1 month ago

I've released https://github.com/nextcloud-releases/integration_google/releases/tag/v3.0.1 with the fix. It should be available on your nextcloud within the hour. Don't hesitate to reach out if you have additional problems or questions :blue_heart: