allow request id to be set. remove reqIdHeaders option #1256

Merged
merged 1 commit into from Dec 22, 2016

Projects

None yet

3 participants

@DonutEspresso
Member
DonutEspresso commented Dec 22, 2016 edited

Fixes #1254

@DonutEspresso DonutEspresso allow request id to be set. remove reqIdHeaders option
ad79f36
@DonutEspresso DonutEspresso requested review from micahr and yunong Dec 22, 2016
@@ -44,8 +43,7 @@ before(function (cb) {
dtrace: helper.dtrace,
handleUncaughtExceptions: true,
log: helper.getLog('server'),
- version: ['2.0.0', '0.5.4', '1.4.3'],
- reqIdHeaders: ['x-req-id-a', 'x-req-id-b']
+ version: ['2.0.0', '0.5.4', '1.4.3']
@DonutEspresso
DonutEspresso Dec 22, 2016 Member

Going to move all these reqIdHeaders tests into plugins repo.

@DonutEspresso DonutEspresso referenced this pull request in restify/plugins Dec 22, 2016
Merged

add reqIdHeaders pre plugin #81

@yunong
yunong approved these changes Dec 22, 2016 View changes
@DonutEspresso DonutEspresso merged commit 4fba68c into 5.x Dec 22, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@DonutEspresso DonutEspresso deleted the req-id branch Dec 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment