URL: https://github.com/SSSD/sssd/pull/24
Title: #24: MAN: Add a manpage for the sssd-secrets responder

jhrozek commented:
"""
On Wed, Sep 28, 2016 at 12:55:34AM -0700, Jakub Hrozek wrote:
> On Mon, Sep 26, 2016 at 01:29:12PM -0700, lslebodn wrote:
> > lslebodn commented on this pull request.
> > 
> > 
> > 
> > > +            The secret URI must begin with 
> > > <filename>/secrets/</filename>.
> > +        </para>
> > +            <variablelist>
> > +                <varlistentry>
> > +                    <term>Listing secrets</term>
> > +                    <listitem>
> > +                    <para>
> > +                        To list the available secrets, send a HTTP GET 
> > request
> > +                        with a trailing slash appended to the container 
> > path.
> > +                    </para>
> > +                    <para>
> > +                        Example:
> > +                        <programlisting>
> > +curl -H "Content-Type: application/json" \
> > +     --unix-socket /var/run/secrets.socket \
> > +     -XGET http://localhost/secrets/
> > 
> > IIRC curl had support for HTTP request via unix sockets even in older 
> > versions
> > But cURL 7.50+ requires a valid URL to be provided, including a hostname.
> > older versions could use `-XGET http:/secrets/`.
> > 
> > we needn't mention that in man page but we can mention requirement  for 
> > "cURL 7.50+"
> 
> But since we include the URL in the examples, that should work with both
> old and new versions, right?

Or does the old curl version not work with a full URL?

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/24#issuecomment-250099261
_______________________________________________
sssd-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to