PUTs to Site Endpoint Requires Value for time_zone #1768

New Issue

adam · 2025-12-29T17:18:57+01:00

adam commented

2025-12-29 17:18:57 +01:00

Originally created by @zachmoody on GitHub (Jun 5, 2018).

Issue type

[ ] Feature request
[X] Bug report
[ ] Documentation

Environment

Python version: 2.6.7
NetBox version: 2.4-dev, but includes previous versions as well.

Description

More details over at digitalocean/pynetbox#59, but when the time_zone field is present and null we get an error saying it can't be null. Omitting the field doesn't return an error.

Originally created by @zachmoody on GitHub (Jun 5, 2018).  ### Issue type [ ] Feature request  [X] Bug report  [ ] Documentation   ### Environment * Python version: 2.6.7 * NetBox version: 2.4-dev, but includes previous versions as well.  ### Description More details over at digitalocean/pynetbox#59, but when the `time_zone` field is present and null we get an error saying it can't be null. Omitting the field doesn't return an error.

adam closed this issue

2025-12-29 17:18:58 +01:00

adam commented

2025-12-29 17:18:58 +01:00

@jeremystretch commented on GitHub (Jun 6, 2018):

I know we talked about this offline, but looking at it again I think it's a question of semantics. The time zone is stored as a string, e.g. America/New_York. Thus, a null value isn't technically valid; an empty value would be represented as an empty string instead. The same error is returned for other CharFields when a null value is passed:

{
    "facility": [
        "This field may not be null."
    ]
}

@jeremystretch commented on GitHub (Jun 6, 2018): I know we talked about this offline, but looking at it again I think it's a question of semantics. The time zone is stored as a string, e.g. `America/New_York`. Thus, a null value isn't technically valid; an empty value would be represented as an empty string instead. The same error is returned for other CharFields when a null value is passed: ``` { "facility": [ "This field may not be null." ] } ```

adam commented

2025-12-29 17:18:58 +01:00

@zachmoody commented on GitHub (Jun 6, 2018):

Empty string works too. The only thing causing pynetbox to go sideways is that GET returns a null. So if the field goes unchanged by the user, it simply gets added as-is to the PUT.

@zachmoody commented on GitHub (Jun 6, 2018): Empty string works too. The only thing causing pynetbox to go sideways is that GET returns a `null`. So if the field goes unchanged by the user, it simply gets added as-is to the PUT.

adam commented

2025-12-29 17:18:58 +01:00

@jeremystretch commented on GitHub (Jun 6, 2018):

The documentation for django-timezone-field states that None should be a valid value for both input and output, so let's stick with your approach.

@jeremystretch commented on GitHub (Jun 6, 2018): The documentation for [`django-timezone-field`](https://github.com/mfogel/django-timezone-field/blob/master/timezone_field/fields.py) states that None should be a valid value for both input and output, so let's stick with your approach.

adam commented

2025-12-29 17:18:58 +01:00

@nue-melexis commented on GitHub (Nov 28, 2018):

Hi,

I still get this issue with version 2.4.3.
Check master commits https://github.com/digitalocean/netbox/commits/master?after=8d4329197a916ed5704ce774a77ec1f59c8a6328+279

Browsing files:
Still OK:
Post-release version bump @jeremystretch committed on Jul 2

NOK:
Closes #2211: Removed Python 2 instructions from the installation docs @jeremystretch committed on Jul 2

@jeremystretch
Can you hint me? (This history for the file at that 2 revisions deviates from March 1st)

Best Regards,
Norman

@nue-melexis commented on GitHub (Nov 28, 2018): Hi, I still get this issue with version 2.4.3. Check master commits https://github.com/digitalocean/netbox/commits/master?after=8d4329197a916ed5704ce774a77ec1f59c8a6328+279 Browsing files: Still OK: [Post-release version bump @jeremystretch committed on Jul 2](https://github.com/digitalocean/netbox/blob/302c14186a2020fa2a241cb0b0bbe1fd9dc0a367/netbox/dcim/api/serializers.py) NOK: [Closes #2211: Removed Python 2 instructions from the installation docs @jeremystretch committed on Jul 2 ](https://github.com/digitalocean/netbox/blob/104bd1b45f6a6f3b5d94fef4126c3b49413b33a3/netbox/dcim/api/serializers.py) @jeremystretch Can you hint me? (This history for the file at that 2 revisions deviates from March 1st) Best Regards, Norman

Sign in to join this conversation.

Branches Tags

main

update-changelog-comments-docs

feature-removal-issue-type

20911-dropdown

20239-plugin-menu-classes-mutable-state

21097-graphql-id-lookups

feature

fix_module_substitution

20923-dcim-templates

20044-elevation-stuck-lightmode

feature-ip-prefix-link

v4.5-beta1-release

20068-import-moduletype-attrs

20766-fix-german-translation-code-literals

20378-del-script

7604-filter-modifiers-v3

circuit-swap

12318-case-insensitive-uniqueness

20637-improve-device-q-filter

20660-script-load

19724-graphql

20614-update-ruff

14884-script

02496-max-page

19720-macaddress-interface-generic-relation

19408-circuit-terminations-export-templates

20203-openapi-check

fix-19669-api-image-download

7604-filter-modifiers

19275-fixes-interface-bulk-edit

fix-17794-get_field_value_return_list

11507-show-aggregate-and-rir-on-api

9583-add_column_specific_search_field_to_tables

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/netbox#1768