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

apply env file #158

Merged
merged 1 commit into from
Nov 24, 2024
Merged

apply env file #158

merged 1 commit into from
Nov 24, 2024

Conversation

XmJwit
Copy link
Contributor

@XmJwit XmJwit commented Nov 7, 2024

No description provided.

Copy link
Owner

@EverythingSuckz EverythingSuckz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @XmJwit,

Thank you for your contribution! However, it seems that the ./fsb.env file is already defined using a volume mapping in the docker-compose.yaml file:

volumes:
  - ./fsb.env:/app/fsb.env

The addition of the env_file section appears to be redundant:

env_file:
  - path: ./fsb.env
    required: true

Could you please review this change and confirm if it is necessary? If there is a specific reason for this addition, please provide more context.

Thank you!

@XmJwit
Copy link
Contributor Author

XmJwit commented Nov 8, 2024

There is a vps, Ubuntu 20.04.6 LTS (GNU/Linux 5.15.0-1035-oracle aarch64), to be install TG-FileStreamBot. Firstly I used the default docker-compose.yaml file to install, but it could not find the fsb.env file! Here is the log I just rerun the config:
PixPin_2024-11-08_21-36-58

so, I try to use the docker command and all is ok :
PixPin_2024-11-08_21-46-41

Finally, I considered that the environment variable configuration file, fsb.env, was not loaded into the container. So I used docker-compose external environment variable file loaded configuration:

env_file:
  - path: ./fsb.env
    required: true

And, as expected, it worked.
image

That's what I added.
All the best! 😄

Copy link
Owner

@EverythingSuckz EverythingSuckz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution and I'm sorry for the delay.

@EverythingSuckz EverythingSuckz merged commit 130d9a8 into EverythingSuckz:main Nov 24, 2024
1 check failed
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 this pull request may close these issues.

2 participants