Code review comment for lp:~diegosarmentero/clickmanager-plugin/x-click-token-support

Revision history for this message
Alejandro J. Cura (alecu) wrote :

179 + } else if (reply->hasRawHeader("X-Click-Token")) {

What's below this line makes no sense at this point of the code flow. If this method is answering for the response to the HEAD request, then the program flow should not even attempt to parse the json.

review: Needs Fixing

« Back to merge proposal