On 9 June 2025 2:20:51 am IST, Junio C Hamano <gitster@xxxxxxxxx> wrote: >Aditya Garg <gargaditya08@xxxxxxxx> writes: > >> Aditya Garg (10): >> imap-send: fix bug causing cfg->folder being set to NULL >> imap-send: fix memory leak in case auth_cram_md5 fails >> imap-send: gracefully fail if CRAM-MD5 authentication is requested >> without OpenSSL >> imap-send: add support for OAuth2.0 authentication >> imap-send: add PLAIN authentication method to OpenSSL >> imap-send: enable specifying the folder using the command line >> imap-send: fix minor mistakes in the logs >> imap-send: display port alongwith host when git credential is invoked >> imap-send: display the destination mailbox when sending a message >> imap-send: add ability to list the available folders > >Very nicely organized, starting from obvious fixes and usability >polishing at the beginning of the series, followed by feature >enhancements to add new variants to the existing framework, followed >by three new features. The "ok"->"OK" change seems somewhat out of >place (I would have done it early if the changes are not controversial, >or very late after the series if the changes looked merely subjective, >and not in the middle either case), but other than that the series is >now organized perfectly. Let's shift it at the last then. > >I think the "fixes" and "auth method enhancements" in the earlier >part are the same as before and I was happy with the resulting code. > >I didn't seriously read the last three or four patches during the >previous round, so I would say they still need reviews, but the >early part of the series now looks very well. > >Thanks. >
