You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2021/04/08 19:20:11 UTC

[GitHub] [trafficcontrol] rawlinp commented on issue #5724: PUT /api/3.x/servers with no xmppId (hashId) results in an internal server error

rawlinp commented on issue #5724:
URL: https://github.com/apache/trafficcontrol/issues/5724#issuecomment-816087364


   Yeah, I think the intent was to return an error to the user if they try to change a field that was made immutable. But in that case, the server update code should assume that a `nil` original XMPPID means that the original XMPPID is the server's `host_name` for comparison purposes.
   
   Also, we should consider adding a DB migration to set nil/empty `xmpp_id` to `host_name`. That is what the CRConfig generation code basically does already.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org