couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jason Smith <...@iriscouch.com>
Subject Re: git commit: Disable checking for string query_params values
Date Mon, 25 Mar 2013 11:09:57 GMT
Oh wow, sorry, all, I missed this.

The commit addresses COUCHDB-1534. However that issue is not yet closed due
to missing unit tests. And Paul wants to remove the function entirely
(which is fine by me).

The comment about iris-monitor was for me, when I diff our branch against
ASF, I would recall the reason for the change. But it is spurious. I
believe I cherry-picked a commit and forgot about the comment.



On Sat, Mar 23, 2013 at 8:02 PM, Noah Slater <nslater@apache.org> wrote:

> Was this a mistaken commit? Was it resolved?
>
>
> On 10 October 2012 21:06, Jan Lehnardt <jan@apache.org> wrote:
>
> > Jason,
> >
> > did you mean to commit this to Apache?
> >
> > Cheers
> > Jan
> > --
> >
> >
> > On Sep 14, 2012, at 04:08 , jhs@apache.org wrote:
> >
> > > Updated Branches:
> > >  refs/heads/COUCHDB-1534 [created] 835b57b19
> > >
> > >
> > > Disable checking for string query_params values
> > >
> > >
> > > Project: http://git-wip-us.apache.org/repos/asf/couchdb/repo
> > > Commit: http://git-wip-us.apache.org/repos/asf/couchdb/commit/835b57b1
> > > Tree: http://git-wip-us.apache.org/repos/asf/couchdb/tree/835b57b1
> > > Diff: http://git-wip-us.apache.org/repos/asf/couchdb/diff/835b57b1
> > >
> > > Branch: refs/heads/COUCHDB-1534
> > > Commit: 835b57b196d423ae5132a559e727031521da8ce9
> > > Parents: 53490d9
> > > Author: Jason Smith (work) <jhs@iriscouch.com>
> > > Authored: Tue Sep 4 06:35:52 2012 +0000
> > > Committer: Jason Smith (air) <jhs@iriscouch.com>
> > > Committed: Fri Sep 14 09:02:04 2012 +0700
> > >
> > > ----------------------------------------------------------------------
> > > src/couchdb/couch_httpd_replicator.erl |    6 ++++--
> > > 1 files changed, 4 insertions(+), 2 deletions(-)
> > > ----------------------------------------------------------------------
> > >
> > >
> > >
> >
> http://git-wip-us.apache.org/repos/asf/couchdb/blob/835b57b1/src/couchdb/couch_httpd_replicator.erl
> > > ----------------------------------------------------------------------
> > > diff --git a/src/couchdb/couch_httpd_replicator.erl
> > b/src/couchdb/couch_httpd_replicator.erl
> > > index fb1e350..47de476 100644
> > > --- a/src/couchdb/couch_httpd_replicator.erl
> > > +++ b/src/couchdb/couch_httpd_replicator.erl
> > > @@ -58,8 +58,10 @@ validate_rep_props([]) ->
> > > validate_rep_props([{<<"query_params">>, {Params}}|Rest]) ->
> > >     lists:foreach(fun
> > >         ({_,V}) when is_binary(V) -> ok;
> > > -        ({K,_}) -> throw({bad_request,
> > > -            <<K/binary," value must be a string.">>})
> > > +        % Disabled to support the iris-monitor replicator
> > > +        %({K,_}) -> throw({bad_request,
> > > +        %    <<K/binary," value must be a string.">>})
> > > +        ({K,_}) -> ok
> > >         end, Params),
> > >     validate_rep_props(Rest);
> > > validate_rep_props([_|Rest]) ->
> > >
> >
> >
>
>
> --
> NS
>



-- 
Iris Couch

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message