diff options
author | Loïc Hoguin <[email protected]> | 2016-08-29 12:39:49 +0200 |
---|---|---|
committer | Loïc Hoguin <[email protected]> | 2016-08-29 12:40:03 +0200 |
commit | c807880f7ac73f813b2660ea81a00f7712a4e793 (patch) | |
tree | ba1d09e9b177f230665a80513b33fbd532000ce4 /archives/extend/2014-September/000464.html | |
parent | b1df25a7d9cda697513650659b781b55b40898f8 (diff) | |
download | ninenines.eu-c807880f7ac73f813b2660ea81a00f7712a4e793.tar.gz ninenines.eu-c807880f7ac73f813b2660ea81a00f7712a4e793.tar.bz2 ninenines.eu-c807880f7ac73f813b2660ea81a00f7712a4e793.zip |
Add old mailing list archives
Diffstat (limited to 'archives/extend/2014-September/000464.html')
-rw-r--r-- | archives/extend/2014-September/000464.html | 89 |
1 files changed, 89 insertions, 0 deletions
diff --git a/archives/extend/2014-September/000464.html b/archives/extend/2014-September/000464.html new file mode 100644 index 00000000..2df89a6e --- /dev/null +++ b/archives/extend/2014-September/000464.html @@ -0,0 +1,89 @@ +<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> +<HTML> + <HEAD> + <TITLE> [99s-extend] Using cowboy_req:body more than once per request + </TITLE> + <LINK REL="Index" HREF="index.html" > + <LINK REL="made" HREF="mailto:extend%40lists.ninenines.eu?Subject=Re%3A%20%5B99s-extend%5D%20Using%20cowboy_req%3Abody%20more%20than%20once%20per%20request&In-Reply-To=%3C54176B4C.1060101%40ninenines.eu%3E"> + <META NAME="robots" CONTENT="index,nofollow"> + <style type="text/css"> + pre { + white-space: pre-wrap; /* css-2.1, curent FF, Opera, Safari */ + } + </style> + <META http-equiv="Content-Type" content="text/html; charset=us-ascii"> + <LINK REL="Previous" HREF="000463.html"> + <LINK REL="Next" HREF="000459.html"> + </HEAD> + <BODY BGCOLOR="#ffffff"> + <H1>[99s-extend] Using cowboy_req:body more than once per request</H1> + <B>Loïc Hoguin</B> + <A HREF="mailto:extend%40lists.ninenines.eu?Subject=Re%3A%20%5B99s-extend%5D%20Using%20cowboy_req%3Abody%20more%20than%20once%20per%20request&In-Reply-To=%3C54176B4C.1060101%40ninenines.eu%3E" + TITLE="[99s-extend] Using cowboy_req:body more than once per request">essen at ninenines.eu + </A><BR> + <I>Tue Sep 16 00:42:20 CEST 2014</I> + <P><UL> + <LI>Previous message: <A HREF="000463.html">[99s-extend] Using cowboy_req:body more than once per request +</A></li> + <LI>Next message: <A HREF="000459.html">[99s-extend] :binding doc +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#464">[ date ]</a> + <a href="thread.html#464">[ thread ]</a> + <a href="subject.html#464">[ subject ]</a> + <a href="author.html#464">[ author ]</a> + </LI> + </UL> + <HR> +<!--beginarticle--> +<PRE>On 09/16/2014 12:35 AM, Paulo F. Oliveira wrote: +><i> Hi. +</I>><i> +</I>>><i> It seems a bit weird to me to read the body and validate it before validating the request itself. +</I>><i> +</I>><i> It certainly seems like it, but I had no immediate solution and +</I>><i> instead of changing a dozen handlers, this seemed faster to implement +</I>><i> :D. I don't understand what you mean by "validating the request +</I>><i> itself". I read the header (I mentioned previously) and the body and +</I>><i> check one against the other. They are present and enough for the +</I>><i> _validator_ to make a decision, but I might be missing something here. +</I> +Like, is it the right method? Are the bindings/qs parameters/headers +present and valid? And so on. The body should be the last thing you +check, due to how expensive it can be, not the first. + +>><i> I would explicitly put these checks in the handler directly. This of course means that there is no need to read it twice anymore. +</I>><i> +</I>><i> I've been trying to find a way to easily share code between handlers +</I>><i> without having to rewrite a lot of code (even if I do decide to put +</I>><i> things in a library function - or several). I recently came across +</I>><i> <A HREF="https://github.com/opscode/mixer.">https://github.com/opscode/mixer.</A> Have you ever used it? +</I> +I usually share code by writing functions. Then I call these functions +where needed. + +-- +Loïc Hoguin +<A HREF="http://ninenines.eu">http://ninenines.eu</A> +</PRE> + +<!--endarticle--> + <HR> + <P><UL> + <!--threads--> + <LI>Previous message: <A HREF="000463.html">[99s-extend] Using cowboy_req:body more than once per request +</A></li> + <LI>Next message: <A HREF="000459.html">[99s-extend] :binding doc +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#464">[ date ]</a> + <a href="thread.html#464">[ thread ]</a> + <a href="subject.html#464">[ subject ]</a> + <a href="author.html#464">[ author ]</a> + </LI> + </UL> + +<hr> +<a href="https://lists.ninenines.eu/listinfo/extend">More information about the Extend +mailing list</a><br> +</body></html> |