Allow number/normal to be used as a method type

[bigdogmat]

Fixes #934

[Divran]

fine by me
@Nebual @AbigailBuccaneer @TomyLobo @syranide

[Nebual]

If it works, sure. I'd prefer to know why it was disabled in the first place though.

[Divran]

Because syranide wanted it to be (read the issue bigdogmat linked to see syranide's reply)

[syranide]

rabble rabble rabble 👍

[thegrb93]

You sure it wasn't put in place to stop some sort of exploit? Just checking.

[TomyLobo]

@thegrb93 seems unlikely. how would that work?

[thegrb93]

I don't know anything about the internals of e2, but usually a feature is excluded like that if there is a problem with it, but Idk.

[brandonsturgeon]

Ship it! See if things break.

[AbigailBuccaneer]

@brandonsturgeon already shipped. I decided @thegrb93 was just talking nonsense and so ignored them :)

[thegrb93]

I'm not talking nonsense. Features are commonly disabled like that due to a problem.

[syranide]

@thegrb93 #934 (comment)

[thegrb93]

"I can't imagine it being there for any other reason" doesn't sound very convincing, but if you guys tested it and there's no obvious exploits then okay.

[Divran]

why are you talking about this. Syranide already confirmed that it was
disabled because he wanted it to be. No other reason. It's not broken.

On Wed, Oct 7, 2015 at 6:15 PM, thegrb93 [email protected] wrote:

"I can't imagine it being there for any other reason" doesn't sound very
convincing, but if you guys tested it and there's no obvious exploits then
okay.

—
Reply to this email directly or view it on GitHub
#977 (comment).

[thegrb93]

Because I'm concerned about the safety of my server as anyone should be.

[syranide]

In the end it's just another (special) parameter, so other than parsing issues there's nothing to worry about.

[immibis]

Can we rename normal to number everywhere yet?

[TomyLobo]

"everywhere" would include code beyond our control