parseMayParam #1

Open
wants to merge 3 commits into
from

Projects

None yet

2 participants

@dzhus

This allows parsing a request parameter using an attoparsec Parser, which I found quite useful. The original name of the routine was getParamWith but I renamed it to match other functions in CoreUtils module.

@mightybyte
Collaborator

Hmmmm, my initial thought is to not pollute the package with dependencies on things like attoparsec. Maybe a more general function for dealing with Maybes and params would be good though.

@dzhus

I thought of this as well. However, aeson already pulls attoparsec into the dep tree.

@dzhus

Le bump.

@dzhus

Any new thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment