Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
L
libvdfp-pipe
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
uplex-varnish
libvdfp-pipe
Commits
cac7d254
Commit
cac7d254
authored
Aug 07, 2020
by
Geoff Simmons
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Bugfix polling for stderr.
parent
56877d54
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
35 additions
and
1 deletion
+35
-1
error.vtc
src/tests/error.vtc
+34
-0
vdfp_pipe.c
src/vdfp_pipe.c
+1
-1
No files found.
src/tests/error.vtc
0 → 100644
View file @
cac7d254
# looks like -*- vcl -*-
varnishtest "vdp error handling"
shell "rm -f ${tmpdir}/foo"
server s1 {
rxreq
txresp -body {foo bar baz quux}
} -start
varnish v1 -vcl+backend {
import ${vmod_pipe};
sub vcl_init {
new cat = pipe.vdp(path="${cat}");
cat.arg("${tmpdir}/foo");
}
sub vcl_deliver {
set resp.filters = "cat";
}
} -start
client c1 {
txreq
rxresp -no_obj
} -run
logexpect l1 -v v1 -g vxid -d 1 -q Error {
expect 0 * Begin {^req \d+ rxreq$}
expect * = Error {^vdfp_pipe: vdp cat: ${cat} stderr}
expect * = End
} -run
src/vdfp_pipe.c
View file @
cac7d254
...
...
@@ -198,7 +198,7 @@ vdp_init(struct req *req, void **priv)
state
->
chldpid
=
-
1
;
state
->
fds
[
STDIN_FILENO
].
events
=
POLLOUT
;
state
->
fds
[
STDOUT_FILENO
].
events
=
POLLIN
;
state
->
fds
[
STDERR_FILENO
].
events
=
POLL
OUT
;
state
->
fds
[
STDERR_FILENO
].
events
=
POLL
IN
;
argv
=
obj
->
argv
;
/* Hackishly using non-API knowledge of VRT_priv_task() */
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment