summaryrefslogtreecommitdiffstats
path: root/archives/extend/2013-October/000292.html
blob: 373e81475c7b3b67ccbb783fb3f74d8229a55d31 (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<HTML>
 <HEAD>
   <TITLE> [99s-extend] REST handler failure
   </TITLE>
   <LINK REL="Index" HREF="index.html" >
   <LINK REL="made" HREF="mailto:extend%40lists.ninenines.eu?Subject=Re%3A%20%5B99s-extend%5D%20REST%20handler%20failure&In-Reply-To=%3CCAJCf5Rx_o66FDE-91kX_%3DKBQou9wcx%2BUEenGPFYLE2nR_PRh8g%40mail.gmail.com%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="000291.html">
   
 </HEAD>
 <BODY BGCOLOR="#ffffff">
   <H1>[99s-extend] REST handler failure</H1>
    <B>Daniel Goertzen</B> 
    <A HREF="mailto:extend%40lists.ninenines.eu?Subject=Re%3A%20%5B99s-extend%5D%20REST%20handler%20failure&In-Reply-To=%3CCAJCf5Rx_o66FDE-91kX_%3DKBQou9wcx%2BUEenGPFYLE2nR_PRh8g%40mail.gmail.com%3E"
       TITLE="[99s-extend] REST handler failure">daniel.goertzen at gmail.com
       </A><BR>
    <I>Wed Oct 30 16:32:47 CET 2013</I>
    <P><UL>
        <LI>Previous message: <A HREF="000291.html">[99s-extend] REST handler failure
</A></li>
        
         <LI> <B>Messages sorted by:</B> 
              <a href="date.html#292">[ date ]</a>
              <a href="thread.html#292">[ thread ]</a>
              <a href="subject.html#292">[ subject ]</a>
              <a href="author.html#292">[ author ]</a>
         </LI>
       </UL>
    <HR>  
<!--beginarticle-->
<PRE>Returning 'halt' caused a status code of 204.

Dan.


On Wed, Oct 30, 2013 at 10:27 AM, Ivan uemlianin &lt;<A HREF="https://lists.ninenines.eu/listinfo/extend">ivan at llaisdy.com</A>&gt; wrote:

&gt;<i> Instead of &lt;&lt;&quot;this body ignored&quot;&gt;&gt; can you return the atom halt?
</I>&gt;<i>
</I>&gt;<i> #dontevenhaveanyofmycodewithme:(
</I>&gt;<i>
</I>&gt;<i> Ivan
</I>&gt;<i>
</I>&gt;<i> --
</I>&gt;<i> festina lente
</I>&gt;<i>
</I>&gt;<i>
</I>&gt;<i> On 30 Oct 2013, at 15:58, Daniel Goertzen &lt;<A HREF="https://lists.ninenines.eu/listinfo/extend">daniel.goertzen at gmail.com</A>&gt;
</I>&gt;<i> wrote:
</I>&gt;<i>
</I>&gt;<i> Well, this sort of works.  I tried this in the response handler:
</I>&gt;<i>
</I>&gt;<i>             {ok, Req2} = cowboy_req:reply(404, [] , &lt;&lt;&quot;this is the body
</I>&gt;<i> that gets used&quot;&gt;&gt;, Req1),
</I>&gt;<i>             {&lt;&lt;&quot;this body gets ignored&quot;&gt;&gt;, Req2, State};
</I>&gt;<i>
</I>&gt;<i>
</I>&gt;<i>
</I>&gt;<i> The client receives a 404 response, but cowboy crashes:
</I>&gt;<i>
</I>&gt;<i> =ERROR REPORT==== 8-Sep-2013::22:22:03 ===
</I>&gt;<i> Error in process &lt;0.131.0&gt; with exit value:
</I>&gt;<i> {function_clause,[{cowboy_req,reply,[200,[],&lt;&lt;31
</I>&gt;<i> bytes&gt;&gt;,{http_req,#Port&lt;0.1208&gt;,ranch_tcp,keepalive,&lt;0.131.0&gt;,&lt;&lt;3
</I>&gt;<i> bytes&gt;&gt;,'HTTP/1.1',{{192,168,1,187},51017},&lt;&lt;12 bytes&gt;&gt;,undefined,80,&lt;&lt;24
</I>&gt;<i> bytes&gt;&gt;,undefined,&lt;&lt;0 bytes&gt;&gt;,undefined,[{channel_num,3}],[{&lt;&lt;10
</I>&gt;<i> bytes&gt;&gt;,&lt;&lt;11 bytes&gt;&gt;},{&lt;&lt;4 bytes&gt;&gt;,&lt;&lt;12 bytes&gt;&gt;},{&lt;&lt;6 bytes&gt;&gt;,&lt;&lt;3
</I>&gt;<i> bytes&gt;&gt;}],[{&lt;&lt;17 bytes&gt;&gt;,undefined},{&lt;&lt;13 bytes&gt;&gt;,undefined},{&lt;&lt;19
</I>&gt;<i> bytes&gt;&gt;,undefined},{&lt;&lt;8 bytes&gt;&gt;,undefined},{&lt;&lt;6 bytes&gt;&gt;,[...
</I>&gt;<i>
</I>&gt;<i>
</I>&gt;<i>
</I>&gt;<i> The issue is that the REST wrapper wants to do the cowboy_req:reply(), and
</I>&gt;<i> when we do the call we cause the wrapper's call to fail.
</I>&gt;<i>
</I>&gt;<i> Dan.
</I>&gt;<i>
</I>&gt;<i>
</I>&gt;<i>
</I>&gt;<i> On Tue, Oct 29, 2013 at 4:01 PM, Ivan uemlianin &lt;<A HREF="https://lists.ninenines.eu/listinfo/extend">ivan at llaisdy.com</A>&gt; wrote:
</I>&gt;<i>
</I>&gt;&gt;<i> Sorry for terse but I only have a phone. Why can't you return a 404 here?
</I>&gt;&gt;<i>  Using something like cowboy:reply(404, ...
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> Ivan
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> --
</I>&gt;&gt;<i> festina lente
</I>&gt;&gt;<i>
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> On 29 Oct 2013, at 21:25, Daniel Goertzen &lt;<A HREF="https://lists.ninenines.eu/listinfo/extend">daniel.goertzen at gmail.com</A>&gt;
</I>&gt;&gt;<i> wrote:
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> My situation is that I have a rest handler that may fail due to invalid
</I>&gt;&gt;<i> url segments.  Example situation:
</I>&gt;&gt;<i>
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> init(_Transport, _Req, _Opts) -&gt;
</I>&gt;&gt;<i>         {upgrade, protocol, cowboy_rest}.
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> content_types_provided(Req, State) -&gt;
</I>&gt;&gt;<i>     {[{&lt;&lt;&quot;application/json&quot;&gt;&gt;, get_json}], Req, State}.
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> get_json(Req0, State) -&gt;
</I>&gt;&gt;<i>     {Params, Req1} = lists:mapfoldl(fun cowboy_req:binding/2, Req0,
</I>&gt;&gt;<i> [param1, param2, param3, ....]),
</I>&gt;&gt;<i>
</I>&gt;&gt;<i>     case catch other_module:request(Params) of
</I>&gt;&gt;<i>         {'EXIT', {badarg, _}} -&gt;
</I>&gt;&gt;<i>             hmmm, Params were bad and I would like to return a 404 code
</I>&gt;&gt;<i> now.
</I>&gt;&gt;<i>         Result -&gt;
</I>&gt;&gt;<i>             {jiffy:encode(Result), Req1, State}
</I>&gt;&gt;<i>     end.
</I>&gt;&gt;<i>
</I>&gt;&gt;<i>
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> So I would like to return a 404 code when my underlying request function
</I>&gt;&gt;<i> fails, but it appears my choices are:
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> - return a 200 (ok) response with data.
</I>&gt;&gt;<i> - crash and cause a 500 (Internal Server Error) response to be returned.
</I>&gt;&gt;<i> Not exactly the sentiment I want.
</I>&gt;&gt;<i>
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> Is there some other way to cause a 404 response?
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> I realize I could add path constraint functions, but I will be
</I>&gt;&gt;<i> replicating logic from my underlying request function.  Furthermore, the
</I>&gt;&gt;<i> constraint functions consider parameters in isolation, so that won't work
</I>&gt;&gt;<i> if the validity of parameters is coupled.
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> Thanks,
</I>&gt;&gt;<i> Dan.
</I>&gt;&gt;<i>
</I>&gt;&gt;<i> _______________________________________________
</I>&gt;&gt;<i> Extend mailing list
</I>&gt;&gt;<i> <A HREF="https://lists.ninenines.eu/listinfo/extend">Extend at lists.ninenines.eu</A>
</I>&gt;&gt;<i> <A HREF="http://lists.ninenines.eu:81/listinfo/extend">http://lists.ninenines.eu:81/listinfo/extend</A>
</I>&gt;&gt;<i>
</I>&gt;&gt;<i>
</I>&gt;<i>
</I>-------------- next part --------------
An HTML attachment was scrubbed...
URL: &lt;<A HREF="http://lists.ninenines.eu/archives/extend/attachments/20131030/0ab7c8ee/attachment.html">http://lists.ninenines.eu/archives/extend/attachments/20131030/0ab7c8ee/attachment.html</A>&gt;
</PRE>

<!--endarticle-->
    <HR>
    <P><UL>
        <!--threads-->
	<LI>Previous message: <A HREF="000291.html">[99s-extend] REST handler failure
</A></li>
	
         <LI> <B>Messages sorted by:</B> 
              <a href="date.html#292">[ date ]</a>
              <a href="thread.html#292">[ thread ]</a>
              <a href="subject.html#292">[ subject ]</a>
              <a href="author.html#292">[ author ]</a>
         </LI>
       </UL>

<hr>
<a href="https://lists.ninenines.eu/listinfo/extend">More information about the Extend
mailing list</a><br>
</body></html>