connman.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* Re: [RFC connman v2 1/1] WIP: Add wpa_supplicant WPA3-SAE support
       [not found] ` <20210527214134.33792-2-ariel.dalessandro@collabora.com>
@ 2021-06-14  7:00   ` Daniel Wagner
  2021-06-14 11:50     ` Ariel D'Alessandro
  0 siblings, 1 reply; 3+ messages in thread
From: Daniel Wagner @ 2021-06-14  7:00 UTC (permalink / raw)
  To: Ariel D'Alessandro
  Cc: connman, iwd, Sven.Dembianny, andrew.shadura, niraj.g,
	n.chaprana, marcel

Hi Ariel,

On Thu, May 27, 2021 at 06:41:34PM -0300, Ariel D'Alessandro wrote:
> Implement WPA3-SAE authentication on connman side using wpa_supplicant
> backend.
> 
> Initially based on Tizen project:
> 
>   https://review.tizen.org/git/?p=platform/upstream/connman.git
> 
> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>

We don't do the SOB, so I just dropped.

I applied the patch, there were small style issues which I fixed it
directly.

Probably we should update the documentation which wpa_supplicant version
is needed to support WPA3.

Thanks,
Daniel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC connman v2 1/1] WIP: Add wpa_supplicant WPA3-SAE support
  2021-06-14  7:00   ` [RFC connman v2 1/1] WIP: Add wpa_supplicant WPA3-SAE support Daniel Wagner
@ 2021-06-14 11:50     ` Ariel D'Alessandro
  2021-06-14 11:57       ` Daniel Wagner
  0 siblings, 1 reply; 3+ messages in thread
From: Ariel D'Alessandro @ 2021-06-14 11:50 UTC (permalink / raw)
  To: Daniel Wagner
  Cc: connman, iwd, Sven.Dembianny, andrew.shadura, niraj.g,
	n.chaprana, marcel

Hi Daniel,

On 6/14/21 4:00 AM, Daniel Wagner wrote:
> Hi Ariel,
> 
> On Thu, May 27, 2021 at 06:41:34PM -0300, Ariel D'Alessandro wrote:
>> Implement WPA3-SAE authentication on connman side using wpa_supplicant
>> backend.
>>
>> Initially based on Tizen project:
>>
>>   https://review.tizen.org/git/?p=platform/upstream/connman.git
>>
>> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> 
> We don't do the SOB, so I just dropped.

Ok.

> 
> I applied the patch, there were small style issues which I fixed it
> directly.

Great! Thanks a lot for your help.

> 
> Probably we should update the documentation which wpa_supplicant version
> is needed to support WPA3.

Which doc files should be updated? I can take a look at that and put
some notes together.

Regards,
Ariel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [RFC connman v2 1/1] WIP: Add wpa_supplicant WPA3-SAE support
  2021-06-14 11:50     ` Ariel D'Alessandro
@ 2021-06-14 11:57       ` Daniel Wagner
  0 siblings, 0 replies; 3+ messages in thread
From: Daniel Wagner @ 2021-06-14 11:57 UTC (permalink / raw)
  To: Ariel D'Alessandro
  Cc: connman, iwd, Sven.Dembianny, andrew.shadura, niraj.g,
	n.chaprana, marcel

Hi Ariel,

On Mon, Jun 14, 2021 at 08:50:10AM -0300, Ariel D'Alessandro wrote:
> > I applied the patch, there were small style issues which I fixed it
> > directly.
> 
> Great! Thanks a lot for your help.

Thanks for your contribution. Really appreciated.

> > Probably we should update the documentation which wpa_supplicant version
> > is needed to support WPA3.
> 
> Which doc files should be updated? I can take a look at that and put
> some notes together.

There was some recommendation on the wpa_supplicant version in the
README. I wonder if this is still valid.

Thanks,
Daniel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-06-14 11:57 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20210527214134.33792-1-ariel.dalessandro@collabora.com>
     [not found] ` <20210527214134.33792-2-ariel.dalessandro@collabora.com>
2021-06-14  7:00   ` [RFC connman v2 1/1] WIP: Add wpa_supplicant WPA3-SAE support Daniel Wagner
2021-06-14 11:50     ` Ariel D'Alessandro
2021-06-14 11:57       ` Daniel Wagner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).