Skip to content
This repository has been archived by the owner on May 17, 2019. It is now read-only.

Set koa proxy flag to true in production env #326

Merged
merged 1 commit into from
Oct 24, 2018
Merged

Set koa proxy flag to true in production env #326

merged 1 commit into from
Oct 24, 2018

Conversation

rajeshsegu
Copy link
Contributor

This would enable koa to use x-forwarded-for header to populate ctx.ip when behind proxy servers like nginx.

@CLAassistant
Copy link

CLAassistant commented Oct 24, 2018

CLA assistant check
All committers have signed the CLA.

@rajeshsegu rajeshsegu changed the title set koa proxy:true in production env Set koa proxy flag to true in production env Oct 24, 2018
@codecov
Copy link

codecov bot commented Oct 24, 2018

Codecov Report

Merging #326 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #326      +/-   ##
==========================================
+ Coverage   93.31%   93.34%   +0.03%     
==========================================
  Files          18       18              
  Lines         434      436       +2     
  Branches       85       85              
==========================================
+ Hits          405      407       +2     
  Misses         16       16              
  Partials       13       13
Impacted Files Coverage Δ
src/server-app.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d88da2...a5f6629. Read the comment docs.

@rajeshsegu rajeshsegu merged commit 0a32a45 into master Oct 24, 2018
@AlexMSmithCA AlexMSmithCA mentioned this pull request Oct 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants