Page MenuHomePhorge

reformat error messages
ClosedPublic

Authored by dhoffend on Apr 2 2020, 2:15 AM.
Tags
None
Referenced Files
F11584716: D1108.diff
Thu, Mar 28, 9:13 AM
Unknown Object (File)
Fri, Mar 22, 11:02 AM
Unknown Object (File)
Fri, Mar 22, 1:04 AM
Unknown Object (File)
Thu, Mar 21, 10:53 PM
Unknown Object (File)
Thu, Mar 21, 4:34 PM
Unknown Object (File)
Wed, Mar 20, 2:19 PM
Unknown Object (File)
Wed, Mar 20, 4:07 AM
Unknown Object (File)
Tue, Mar 19, 2:33 PM
Subscribers

Details

Summary

Send correct error messages depending on the type of autodiscover method. Microsoft == XML Error Message, JSON == json formatted error message. Sending a generic error 500 is harder to debug in the end.

Diff Detail

Repository
rAC autoconf
Branch
error-codes
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 29017
Build 10519: arc lint + arc unit

Event Timeline

dhoffend created this revision.
machniak requested changes to this revision.Apr 5 2020, 8:26 AM
machniak subscribed.
machniak added inline comments.
lib/AutodiscoverJson.php
124

I guess it should be ErrorCode.

This revision now requires changes to proceed.Apr 5 2020, 8:26 AM
  • switch to an error code a real ms server sends out on invalid request

You're right. I've fixed the typo and replaced the msxml errorcode with one that a real exchange server sends outs. Just in case something weird could happing.

  • one more typo in json protocol
  • change error-id to the an integer
  • always send unsigned int32
This revision was not accepted when it landed; it landed in state Needs Review.Sep 19 2023, 12:03 PM
Closed by commit rACc526e21729a7: reformat error messages (authored by dhoffend, committed by machniak). · Explain Why
This revision was automatically updated to reflect the committed changes.