Comments (5)
I guess the only way to properly solve this is to use the following ES6 technique, following this Stack Overflow post:
http://stackoverflow.com/questions/20147081/javascript-catch-access-to-property-of-object
and
http://wiki.ecmascript.org/doku.php?id=harmony%3adirect_proxies
Edit: The sad thing is that it's not supported by Node.JS at the moment. We might want to take a look into Babel's way of translating this feature; so that we can do it on our own in Arsenal, without relying on babel.
Edit: Even sadder... support cannot be brought through observing Babel:
Unsupported feature
Due to the limitations of ES5, Proxies cannot be transpiled or polyfilled. See support in various JavaScript engines.
I wanna cry :'(
from arsenal.
😞 that hurts.
We wouldn't want to mess with Object's prototype chain either.
from arsenal.
I'm completely not against such kind of metaprogramming, and that's the only way I see that would have allowed to keep the current API, while ensuring that using an undefined error name would have used either a "default error" or failed early.
That might be my dark metaprogrammer side talking here, but I love simple APIs that are powerful :D
from arsenal.
This is where static analysis is supposed to shine. I'll try to look into that in my free time.
from arsenal.
Indeed, it's yet another track to follow, but still, this means setting up a tool in all projects to ensure that this one is used properly. I'd rather have it self contained with black magic ^^'
from arsenal.
Related Issues (20)
- multiple high and critical issues in `yarn audit` HOT 1
- empty/invalid/unredable/none Date header: errors incompatible with AWS
- invalid amz-date header: errors incompatible with AWS
- incorrect authorization header: errors incompatible with AWS
- authv4 sort query params HOT 1
- COMPAT: Increase Signature V2 Expires param
- COMPAT: return AccessDenied if Date before epochTime (01/01/1970) HOT 4
- callApiMethod in routes is not defined
- Error messages should not have periods
- Review of https://github.com/scality/Arsenal/pull/2152
- Re-review of https://github.com/scality/Arsenal/pull/2152
- Policy evaluation, action should be case insensitive
- Tests for #240 HOT 1
- delimiter.js maxKeys==0 param issue HOT 4
- Logs are broken HOT 6
- Exception occurs when using v4 authentication with certain query parameters
- Unbalanced log
- non monotonic clock used in version ID generation
- Action required: Greenkeeper could not be activated 🚨
- Invalid Greenkeeper configuration file
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from arsenal.