Feature request: Support Access-Control-Expose-Headers

[hinok]

It would be great if we could define custom headers that browsers are allowed to access in config/cors.js
It's all about Access-Control-Expose-Headers.

I forked and added very basic implementation.
If you want I can make PR.

config/cors.js looks now

module.exports.cors = {

  // Stripped code for readability

  /***************************************************************************
  *                                                                          *
  * Which headers should be allowed for CORS requests? This is only used in  *
  * response to preflight requests.                                          *
  *                                                                          *
  ***************************************************************************/

  headers: 'content-type, Total-Count',

  /***************************************************************************
  *                                                                          *
  * Which headers in CORS response should be allowed for borwser? This is    *
  * only used in response to preflight requests.                             *
  *                                                                          *
  ***************************************************************************/

  exposeHeaders: 'Total-Count'

};

My solution: hinok@81ed85e

@gaprl mentioned that earlier: #2146

[loicsaintroch]

I like your idea @hinok. We would love you to write a pull request of course! Thank you for helping us making Sails a better framework.

[aantipov]

+1

[artworkad]

@loicsaintroch #2712

[hinok]

@artworkad Good, I didn't have time to write tests for that feature but I'm glad that someone made PR ;)