Re: [DISCUSS] BEARER authentication support in Avatica - 2nd try

2023-12-04 Thread Julian Hyde
I saw the discussion; it was not marked as spam. I am not familiar with BEARER 
authentication so I didn’t reply. If it is indeed a common requirement, and 
people are prepared to do the work, I would be delighted to have this feature.

Julian


> On Dec 3, 2023, at 9:45 PM, Istvan Toth  wrote:
> 
> Thanks Benchao Li!
> 
> You are right, but most people use GMail as their mail client, and having
> the email correctly sent by the list server, and being present in the
> archives amounts to little if most list members don't see it in their inbox.
> I am glad that it wasn't marked spam for everyone. GMail's ways are
> mysterious, I could only react based on what mine did.
> 
> best regards
> Istvan
> 
> 
> On Sun, Dec 3, 2023 at 3:51 PM Benchao Li  wrote:
> 
>> The original email is already in the Apache mailing list server[1].
>> It's the email app's behavior to mark it as spam, at least in my
>> Gmail, it's not spam.
>> 
>> [1] https://lists.apache.org/thread/hggbfwnlmg9hf00n5w68fsw519fzzxd3
>> 
>> Istvan Toth  于2023年12月3日周日 00:38写道:
>>> 
>>> Hi !
>>> 
>>> Aron has recently started a discussion, but Gmail has decided to mark it
>> as
>>> spam.
>>> I expect that most people haven't even seen it, so I'm re-sending it with
>>> the hyperlinks removed:
>>> 
>>> I've opened a jira regarding the addition of Bearer authentication
>> support
 to Avatica CALCITE-6135.
 This feature would enable authentication with bearer tokens as defined
 in RFC 6750.
>>> 
>>> 
 I've also opened a PR with a proposed implementation (link int the
 ticket). Worth noting that
 there is an upcoming support for this auth scheme in HttpClient as well
 (the commit is "BEARER auth scheme support (RFC 6750)")
 Taking this into consideration, the relevant classes are adjusted so a
 future version upgrade should not be too difficult. Additionally, this
 implementation provides greater flexibility with a file based token
 provider allowing token refresh, however, a constant token provider
>> class
 is also implemented based on Istvan's suggestion.
>>> 
>>> 
 I welcome any feedback or suggestions you may have.
>>> 
>>> 
 Regards,
 Áron
>> 
>> 
>> 
>> --
>> 
>> Best,
>> Benchao Li
>> 
> 
> 
> -- 
> *István Tóth* | Sr. Staff Software Engineer
> *Email*: st...@cloudera.com
> cloudera.com 
> [image: Cloudera] 
> [image: Cloudera on Twitter]  [image:
> Cloudera on Facebook]  [image: Cloudera
> on LinkedIn] 
> --
> --



Re: [DISCUSS] BEARER authentication support in Avatica - 2nd try

2023-12-03 Thread Istvan Toth
Thanks Benchao Li!

You are right, but most people use GMail as their mail client, and having
the email correctly sent by the list server, and being present in the
archives amounts to little if most list members don't see it in their inbox.
I am glad that it wasn't marked spam for everyone. GMail's ways are
mysterious, I could only react based on what mine did.

best regards
Istvan


On Sun, Dec 3, 2023 at 3:51 PM Benchao Li  wrote:

> The original email is already in the Apache mailing list server[1].
> It's the email app's behavior to mark it as spam, at least in my
> Gmail, it's not spam.
>
> [1] https://lists.apache.org/thread/hggbfwnlmg9hf00n5w68fsw519fzzxd3
>
> Istvan Toth  于2023年12月3日周日 00:38写道:
> >
> > Hi !
> >
> > Aron has recently started a discussion, but Gmail has decided to mark it
> as
> > spam.
> > I expect that most people haven't even seen it, so I'm re-sending it with
> > the hyperlinks removed:
> >
> > I've opened a jira regarding the addition of Bearer authentication
> support
> > > to Avatica CALCITE-6135.
> > > This feature would enable authentication with bearer tokens as defined
> > > in RFC 6750.
> >
> >
> > > I've also opened a PR with a proposed implementation (link int the
> > > ticket). Worth noting that
> > > there is an upcoming support for this auth scheme in HttpClient as well
> > > (the commit is "BEARER auth scheme support (RFC 6750)")
> > > Taking this into consideration, the relevant classes are adjusted so a
> > > future version upgrade should not be too difficult. Additionally, this
> > > implementation provides greater flexibility with a file based token
> > > provider allowing token refresh, however, a constant token provider
> class
> > > is also implemented based on Istvan's suggestion.
> >
> >
> > > I welcome any feedback or suggestions you may have.
> >
> >
> > > Regards,
> > > Áron
>
>
>
> --
>
> Best,
> Benchao Li
>


-- 
*István Tóth* | Sr. Staff Software Engineer
*Email*: st...@cloudera.com
cloudera.com 
[image: Cloudera] 
[image: Cloudera on Twitter]  [image:
Cloudera on Facebook]  [image: Cloudera
on LinkedIn] 
--
--


Re: [DISCUSS] BEARER authentication support in Avatica - 2nd try

2023-12-03 Thread Benchao Li
The original email is already in the Apache mailing list server[1].
It's the email app's behavior to mark it as spam, at least in my
Gmail, it's not spam.

[1] https://lists.apache.org/thread/hggbfwnlmg9hf00n5w68fsw519fzzxd3

Istvan Toth  于2023年12月3日周日 00:38写道:
>
> Hi !
>
> Aron has recently started a discussion, but Gmail has decided to mark it as
> spam.
> I expect that most people haven't even seen it, so I'm re-sending it with
> the hyperlinks removed:
>
> I've opened a jira regarding the addition of Bearer authentication support
> > to Avatica CALCITE-6135.
> > This feature would enable authentication with bearer tokens as defined
> > in RFC 6750.
>
>
> > I've also opened a PR with a proposed implementation (link int the
> > ticket). Worth noting that
> > there is an upcoming support for this auth scheme in HttpClient as well
> > (the commit is "BEARER auth scheme support (RFC 6750)")
> > Taking this into consideration, the relevant classes are adjusted so a
> > future version upgrade should not be too difficult. Additionally, this
> > implementation provides greater flexibility with a file based token
> > provider allowing token refresh, however, a constant token provider class
> > is also implemented based on Istvan's suggestion.
>
>
> > I welcome any feedback or suggestions you may have.
>
>
> > Regards,
> > Áron



-- 

Best,
Benchao Li


[DISCUSS] BEARER authentication support in Avatica - 2nd try

2023-12-02 Thread Istvan Toth
Hi !

Aron has recently started a discussion, but Gmail has decided to mark it as
spam.
I expect that most people haven't even seen it, so I'm re-sending it with
the hyperlinks removed:

I've opened a jira regarding the addition of Bearer authentication support
> to Avatica CALCITE-6135.
> This feature would enable authentication with bearer tokens as defined
> in RFC 6750.


> I've also opened a PR with a proposed implementation (link int the
> ticket). Worth noting that
> there is an upcoming support for this auth scheme in HttpClient as well
> (the commit is "BEARER auth scheme support (RFC 6750)")
> Taking this into consideration, the relevant classes are adjusted so a
> future version upgrade should not be too difficult. Additionally, this
> implementation provides greater flexibility with a file based token
> provider allowing token refresh, however, a constant token provider class
> is also implemented based on Istvan's suggestion.


> I welcome any feedback or suggestions you may have.


> Regards,
> Áron