Users should default to timezone configured in api.php

[samvasko]

Bug Report

Default timezone for users is 'America/New_York'.

Expected Behavior

The users inherit the timezone from the server api configuration.

Actual Behavior

Timezone is set to 'America/New_York'.

Other Context & Screenshots

Relevant code for admin creation:

https://github.com/directus/api/blob/368684beeb226c1b33af72619abf68ed7631e55a/src/core/Directus/Util/Installation/InstallerUtils.php#L251

Schema configuring the database to default to NYC
https://github.com/directus/api/blob/6898e21872f611be3b9f642c920587a9e707acdc/src/schema.sql#L563

Technical Details

• Device: Desktop
• OS: MacOs High Sierra
• Web Server: nginx
• PHP Version: 7.2
• Database: Mysql 5.2
• Install Method: docker 2.0.17 and 7.0.17

[benhaynes]

@wellingguzman @rijkvanzanten — is this App or API? I guess we want to have users created directly in the API to follow this too.

[wellingguzman]

@benhaynes This is API.

Hey @samvasko to make it clear when you refer to the server timezone you referring to the actual server timezone, and not the php default timezone on the server (php.ini), correct?

[wellingguzman]

Based on the title you are talking about the timezone in the project configuration to be used instead of the static America/New_York.

In other hand the schema.sql file will be always set to America/New_York, (reads cannot be configured) we generated this file from the migrations files.

[samvasko]

I would expect the default timezone to be the one in api.php. I that one is not configured then it should be the system one. 🤷‍♂️

You are right. I thought that the .sql was the source. The migration contains New York as well.
https://github.com/directus/api/blob/368684beeb226c1b33af72619abf68ed7631e55a/migrations/db/schemas/20180220023248_create_users_table.php#L64

[samvasko]

I guess the options would be to either:

1. Read the config file during migration and admin creation
2. Default to UTC

[wellingguzman]

You are right. I thought that the .sql was the source. The migration contains New York as well.

Correct, but we can configure that to fetch the default timezone before creating the project.

I guess the options would be to either:

1. Read the config file during migration and admin creation
2. Default to UTC

Exactly, fallback to something when the default timezone is empty.

[samvasko]

🍾