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.

Sign up for GitHub

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

Jump to bottom

api token page - server error if user is not selected in dropdown box ( should be a required field ) #13433

Closed
ITJamie opened this issue Aug 10, 2023 · 0 comments
Closed

api token page - server error if user is not selected in dropdown box ( should be a required field ) #13433

ITJamie opened this issue Aug 10, 2023 · 0 comments
Assignees
@jeremystretch
Labels
beta Concerns a bug/feature in a beta release status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application

Comments

@ITJamie
Copy link
Contributor

ITJamie commented Aug 10, 2023

NetBox version

3.6-beta1

Python version

3.10

Steps to Reproduce

on beta.
goto admins > api tokens > add token
change nothing on field (don't select user)
press submit

Expected Behavior

validation error expected (or unable to press submit until user is chosen)

Observed Behavior

server error response

<class 'django.db.utils.IntegrityError'>

null value in column "user_id" of relation "users_token" violates not-null constraint
DETAIL:  Failing row contains (3, 2023-08-10 12:25:39.308525+00, null, fa8d61d6903e2dbd747165228b03e4292f72f38b, t, , null, {}, null).

Python version: 3.10.12
NetBox version: 3.6-beta1
@ITJamie ITJamie added the type: bug A confirmed report of unexpected behavior in the application label Aug 10, 2023
@abhi1693 abhi1693 added the status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation label Aug 10, 2023
@jeremystretch jeremystretch added the beta Concerns a bug/feature in a beta release label Aug 10, 2023
@jeremystretch jeremystretch self-assigned this Aug 10, 2023
@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation labels Aug 10, 2023
jeremystretch added a commit that referenced this issue Aug 10, 2023
@jeremystretch
Fixes #13433: User field on API token form should be required
4d2ef0a
jeremystretch added a commit that referenced this issue Aug 10, 2023
@jeremystretch
Changelog for #13433, #13434, #13437
1ff1b4d
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jeremystretch jeremystretch

Labels
beta Concerns a bug/feature in a beta release status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ITJamie @abhi1693 @jeremystretch