Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: AIP-193 – Errors #3

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat: AIP-193 – Errors #3

wants to merge 3 commits into from

Commits on Sep 3, 2020

  1. feat: AIP-193

    This adds a generic AIP for errors. There is probably a decent bit
    for us to discuss here.
    
    Some high-level notes:
    
    - I decided to represent expected JSON interfaces using TypeScript
      (rather than JSONSchema), which I perceive to be much better
      for human readability.
    - We should discuss/debate the proposed Error interface. It is
      mostly similar to what Google uses but with two fields "promoted"
      (we have a huge _mea culpa_ here).
    - I did not discuss any common _headers_ related to error handling
      (e.g. `Retry-After`). I personally think that `Retry-After` gets
      covered in AIP-194, and I could not think of any others that
      warranted inclusion here.
    
    I expect this is an area where everyone will need to make changes,
    but I also notice that entire sections can probably be adopted
    by everyone (e.g. "Messages"), so I think this should work
    reasonably well.
    
    Looking forward to the discussion on this.
    Luke Sneeringer committed Sep 3, 2020
    Configuration menu
    Copy the full SHA
    2ee6e9b View commit details
    Browse the repository at this point in the history

Commits on Sep 28, 2020

  1. Merge branch 'master' into aip-193

    Luke Sneeringer authored Sep 28, 2020
    Configuration menu
    Copy the full SHA
    0908529 View commit details
    Browse the repository at this point in the history

Commits on Nov 4, 2021

  1. Merge branch 'main' into aip-193

    Luke Sneeringer authored Nov 4, 2021
    Configuration menu
    Copy the full SHA
    568e0fc View commit details
    Browse the repository at this point in the history