GNU bug report logs - #72358
29.4; oauth2.el improvements

Previous Next

Package: emacs;

Reported by: Xiyue Deng <manphiz <at> gmail.com>

Date: Tue, 30 Jul 2024 02:20:01 UTC

Severity: normal

Found in version 29.4

Done: Philip Kaludercic <philipk <at> posteo.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Philip Kaludercic <philipk <at> posteo.net>
To: Xiyue Deng <manphiz <at> gmail.com>
Cc: bjorn.bidar <at> thaodan.de, Eli Zaretskii <eliz <at> gnu.org>, fitzsim <at> fitzsim.org, rpluim <at> gmail.com, 72358 <at> debbugs.gnu.org
Subject: bug#72358: 29.4; oauth2.el improvements
Date: Wed, 21 Aug 2024 19:42:28 +0000
Xiyue Deng <manphiz <at> gmail.com> writes:

> Eli Zaretskii <eliz <at> gnu.org> writes:
>
>>> From: Xiyue Deng <manphiz <at> gmail.com>
>>> Cc: Thomas Fitzsimmons <fitzsim <at> fitzsim.org>,  Björn Bidar
>>>  <bjorn.bidar <at> thaodan.de>,  rpluim <at> gmail.com,  72358 <at> debbugs.gnu.org
>>> Date: Wed, 14 Aug 2024 01:23:19 -0700
>>> 
>>> >> It's been a few days since the last time I received feedback for
>>> >> improvements regarding my patches.  Is there any other feedbacks/reviews
>>> >> I am expecting from the co-maintainers?  Please also let me know when
>>> >> it's time to ask for merging and requesting a new tagged release.
>>> >
>>> > ?? The last message in this discussion was just yesterday evening, and
>>> > my understanding is that you are still discussing the issues and did
>>> > not reach the final conclusion.  If I'm mistaken, my apologies;
>>> 
>>> The recent communication was not related to my patches but to check
>>> whether it is possible to support outlook.com OAuth2 login (and the
>>> conclusion was no because refreshing access token was disabled as
>>> confirmed by MS representative during an online chat.)
>>> 
>>> > please describe your conclusion and post the patch that you-all agree
>>> > would solve the issues, and let's take it from there.
>>> 
>>> I actually only received comments from Robert and I have updated my
>>> patches according in [1][2] (also attached in EOM).
>>> 
>>> [1] https://debbugs.gnu.org/cgi/bugreport.cgi?bug=72358#20
>>> [2] https://debbugs.gnu.org/cgi/bugreport.cgi?bug=72358#44
>>
>> Thanks.
>>
>> Philip, could you please DTRT here?  This seems to be an ELPA package.
>
> Friendly ping.  Please also let me know if there are more review
> comments.

I'm sorry, this was a rather long thread and I didn't have the time yet
to follow up on it.  Can you confirm that you want me to review the
patches attached to this message:

  https://debbugs.gnu.org/cgi/bugreport.cgi?bug=72358#20

?

-- 
	Philip Kaludercic on peregrine




This bug report was last modified 257 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.