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

Rest API in Docker #996

Closed
rlljorge opened this issue Oct 2, 2024 · 4 comments · Fixed by #997
Closed

Rest API in Docker #996

rlljorge opened this issue Oct 2, 2024 · 4 comments · Fixed by #997
Labels
Milestone

Comments

@rlljorge
Copy link

rlljorge commented Oct 2, 2024

I am using the Docker version tag 1.7, and the Apache server is missing a configuration:

Alias /rest /var/www/rest

After manually adding the Alias, another issue appears in an include:

PHP Fatal error: Uncaught Error: Failed opening required '../../lib/detectbrowserlanguage.php' (include_path='.:/usr/local/lib/php') in /var/www/rest/v1/include.php:24\nStack trace:\n#0 /var/www/rest/v1/checkpassword.php(2): require_once()\n#1 {main}\n thrown in /var/www/rest/v1/include.php on line 24

@coudot coudot added the bug label Oct 3, 2024
@coudot coudot added this to the 1.7.1 milestone Oct 3, 2024
@coudot
Copy link
Member

coudot commented Oct 3, 2024

Thanks for the report, we will look at it

@coudot
Copy link
Member

coudot commented Oct 3, 2024

There is a fix in #997 for the PHP Fatal error

Regarding the vhost configuration, the Alias seem present but the location is disabled by default. How did you change the configuration?

@rlljorge
Copy link
Author

rlljorge commented Oct 3, 2024

Hello,

I simply added the following configuration:

Alias /rest/ /var/www/rest/

In the file /etc/apache2/sites-enabled/000-default.conf

@coudot
Copy link
Member

coudot commented Oct 29, 2024

Issue with language detection is fixed.

Enabling REST services by default in docker image does not seems a good idea. REST endpoints should be protected at least by a basic auth.

@coudot coudot closed this as completed Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants