Differential D1000

Document default exchange isn't specified in the config
ClosedPublic
Actions

Authored by dereckson on Jul 6 2017, 20:09.

Details

Reviewers

dereckson

Commits

rNDa47a689f337a: Document default exchange isn't specified in the config

Test Plan

Post to /delivery/register_consumer the following payload:

{
	"anything": "anything"
}

(Flask considers request.json doesn't exist if the payload is empty,
this isn't a problem as we don't have any method accepting empty
payloads, as an API key will be required for register_consumer)

Get the following output to the console:

Client wants the default exchange, unspecified in config.
127.0.0.1 - - [06/Jul/2017 22:09:59] "POST /delivery/register_consumer HTTP/1.1" 400 -

Diff Detail

Repository

rND Delivery API for Notifications center

Lint

Lint Passed

Unit

No Test Coverage

Branch

mandatory_default_exchange

Build Status

Buildable 1561
Build 1809: arc lint + arc unit

Event Timeline

dereckson created this revision.Jul 6 2017, 20:09

Harbormaster completed remote builds in B1561: Diff 2555.Jul 6 2017, 20:09

dereckson accepted this revision.Jul 6 2017, 20:09

dereckson edited the test plan for this revision. (Show Details)

This revision is now accepted and ready to land.Jul 6 2017, 20:10

Closed by commit rNDa47a689f337a: Document default exchange isn't specified in the config (authored by dereckson, committed by dereckson). · Explain WhyJul 6 2017, 20:10

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

app.py

1 line

Diff 2555

View Options

Document default exchange isn't specified in the configClosedPublicActions