Ao fazer uma solicitação de API para a rota /posts.json com a propriedade external_id definida para o external_id de um tópico existente, um erro semelhante a este é acionado:
ActiveRecord::RecordNotUnique (PG::UniqueViolation: ERROR: duplicate key value violates unique constraint \"index_topics_on_external_id\" DETAIL: Key (external_id)=(obCopying text to the system clipboard) already exists.)
Quando a solicitação é feita com a gema Discourse API, a resposta é apenas uma string HTML:
#<DiscourseApi::Error:\"\u003c!DOCTYPE html\u003e\n\u003chtml lang=\"en\"\u003e\n\u003chead\u003e\n \u003cmeta charset=\"utf-8\" /\u003e\n ...
Se a solicitação for feita de outra forma, digamos com HTTParty, o objeto de resposta se parece com isto:
response = HTTParty.post(url, body:, headers:)
add_note_to_db(title, response)
# Tenta obter o objeto `response` da resposta
p response.response
# saídas:
# #<Net::HTTPInternalServerError 500 Internal Server Error readbody=true>
Não há muito que possa ser feito com isso.
Aqui estão as chamadas que acionam o problema:
lib/topic_creator.rb:237:in `save_topic'
lib/topic_creator.rb:58:in `create'
lib/post_creator.rb:490:in `create_topic'
lib/post_creator.rb:190:in `block in create'
lib/post_creator.rb:390:in `block in transaction'
lib/post_creator.rb:390:in `transaction'
lib/post_creator.rb:188:in `create'
lib/new_post_manager.rb:318:in `perform_create_post'
lib/new_post_manager.rb:252:in `perform'
app/controllers/posts_controller.rb:210:in `block in create'
lib/distributed_memoizer.rb:16:in `block in memoize'
lib/distributed_mutex.rb:53:in `block in synchronize'
lib/distributed_mutex.rb:49:in `synchronize'
lib/distributed_mutex.rb:49:in `synchronize'
lib/distributed_mutex.rb:34:in `synchronize'
lib/distributed_memoizer.rb:12:in `memoize'
app/controllers/posts_controller.rb:209:in `create'
Eu acho que um erro precisa ser levantado em TopicCreator#create, resgatado do bloco de transação em PostCreator#create, e então tratado no controller de posts.
A solução ideal do meu ponto de vista seria retornar um erro que indicasse qual tópico estava usando o ID externo.
Levante um erro em TopicCreator#create:
def create
topic = Topic.new(setup_topic_params)
if topic.external_id.present? && Topic.with_deleted.exists?(external_id: topic.external_id)
existing_topic = Topic.with_deleted.find_by(external_id: topic.external_id)
raise DuplicateExternalIdError.new(existing_topic), "External ID must be unique. Existing topic ID: #{existing_topic.id}"
end
#...
# Provavelmente existe uma classe de erro existente que poderia lidar com isso, mas colocar isso no final da classe topic_creator funciona para mim:
class DuplicateExternalIdError < StandardError
attr_reader :topic
def initialize(topic)
@topic = topic
end
end
Resgate-o em PostCreator#create:
def create
if valid?
begin
transaction do
build_post_stats
create_topic # é aqui que o erro está acontecendo
# ...
end
rescue TopicCreator::DuplicateExternalIdError => e
@errors.add(:base, "External ID must be unique. Existing topic ID: #{e.topic.id}")
end
end
Trate-o no controller de posts:
# posts_controller.rb
def handle_duplicate_external_id_error(exception)
render json: { error: "External ID must be unique. Existing topic ID: #{exception.topic.id}" }, status: :unprocessable_entity
end
Isso talvez seja muito específico para o meu caso de uso para ser a solução correta. Está relacionado ao problema com IDs externos e tópicos duplicados que é mencionado aqui: API topic's external_ID can't be reused after deleting a topic and creating a new one - #2 by simon. Idealmente, um aplicativo que está postando para a API do Discourse poderia usar a mensagem de resposta para desativar e atualizar o tópico que já estava usando o ID externo.