User API keys: duplicate client_id will lead to internal server error

sam · September 6, 2024, 6:37am

The error should be better, yes, but client_id needs to be unique.

discourse/discourse/blob/ed7ba4b2a749ad26129e4fd848f39a0c4defed33/app/controllers/user_api_keys_controller.rb#L67-L74


      
          key =
            UserApiKey.create!(
              application_name: @application_name,
              client_id: params[:client_id],
              user_id: current_user.id,
              push_url: params[:push_url],
              scopes: scopes.map { |name| UserApiKeyScope.new(name: name) },
            )

When you are sending users that way you must generate a unique id in your API call. The index is correct, 1 user may have N client ids.

Topic		Replies	Views
User API keys specification Developer Guides rest-api , reference	65	31043	June 1, 2024
Generating User Api Keys with REST API Dev rest-api	20	8402	April 8, 2018
Unhandled PG::UniqueViolation for topic external_id field Bug pr-welcome	2	136	June 11, 2024
Unicode usernames Dev rest-api	23	1156	September 11, 2023
Creating new User error after upgrade PG::UniqueViolation Duplicate Key SSO rest-api	2	706	January 24, 2023

User API keys: duplicate client_id will lead to internal server error

Related topics