#377 access control expose headers

atuttle / Taffy

:candy: The REST Web Service framework for ColdFusion and Lucee

http://taffy.io

Other

226 stars 118 forks source link

#377 access control expose headers #379

Closed timmixell closed 4 years ago

timmixell commented 5 years ago

Resolves #377

@atuttle this is all i was envisioning for this one. i'm not 100% what you mean re: config and/or metadata, but i figured you could furnish additional feedback against the PR.

thanks!

pfreitag commented 4 years ago

@timmixell would it be possible to include a test case for this so we can be sure it is working and doesn't get broken in the future?

timmixell commented 4 years ago

@pfreitag per my original comment, I was hoping the proposed changes would solicit some feedback around the additions (either a 👍 or 👎 ). I'll add a test case ASAP.

timmixell commented 4 years ago

@pfreitag sorry for delay in unit tests on this one. should be good to go now.

atuttle commented 4 years ago

Please add documentation for this new configuration option. As in #381 you should assume this will be part of version 3.3.0.

timmixell commented 4 years ago

Please 👀 at your convenience: https://github.com/atuttle/TaffyDocs/pull/19

Thanks!