Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Wrong order in RuleChain and RuleRace evaluation

See original GitHub issue

Describe the bug

Here: https://github.com/maticzav/graphql-shield/blob/master/src/rules.ts#L395 And here: https://github.com/maticzav/graphql-shield/blob/master/src/rules.ts#L449

The concatenation is wrong, it will in reverse order compared to the order of the rules potentially leading to some unwanted behaviours.

They should be replaced by [res, ...res] to keep the order of the rules.

Issue Analytics

State:
Created 3 years ago
Comments:7 (2 by maintainers)

Top GitHub Comments

2reactions

stale[bot]commented, Jul 29, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

1reaction

Syttencommented, Sep 21, 2020

Not stale.

Top Results From Across the Web

RuleChain (JUnit API)

The RuleChain rule allows ordering of TestRules. You create a RuleChain with outerRule(TestRule) and subsequent calls of around(TestRule) :

Untitled

Cma canada entrance exam results 2011, Catamania nantes, Predator mask 1987, ... History of illuminati new world order, Musitano catering cornwall, ...

Untitled

Bairiyaa ramaiya vastavaiya mp3, Sweet earth seitan review, Recubrimiento acrilico ... Optus zoo internet recharge, Gudako fate grand order, Si makhlof, ...

Untitled

... train rule race tumblr raw master. sissy videos in porn star massage ... olson a cheat porn exchange order fart dub school...