Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] Added support for proxying managesieve protocol

Sander Hoentjen
April 16, 2020 01:52PM
Hi Maxim,

Thanks for your response! I will wait till you have time to properly
review my code. That is more important than getting it in fast.

The code is available at
https://github.com/AntagonistHQ/nginx/tree/sieve_v2, so at least it is
out in the public for anyone interested.

For now, I'll just wait and see what will happen in May.

Thank you,

Sander


On 4/15/20 8:56 PM, Maxim Konovalov wrote:
> Hi Sander,
>
> First of all, thanks for your code contribution and your work done on it.
>
> Among with other tasks we are now in a preparation for a new development
> branch 1.19 and don't have much developers resources to make a proper
> review of the code.
>
> I'd suggest to put the module somewhere on external repo for now, e.g.
> on github. I hope we'll be able to return to this topic later in May.
>
> Thanks,
>
> Maxim
>
> On 14.04.2020 21:27, Sander Hoentjen wrote:
>> Hello list,
>>
>> Since the Nginx development procedure is unknown to me: Did i do the
>> right things to get my submission to be considered? What are the next
>> steps? Will somebody review this, or reject it? Or is it possible that
>> it just won't get any attention, and that this will mean it will not be
>> considered? I hope I will at least get some feedback, even if it is a
>> rejection :)
>>
>> Kind regards,
>>
>> Sander
>>
>> On 4/8/20 8:33 PM, Sander Hoentjen wrote:
>>> Hello list,
>>>
>>> This is my attempt at adding support for the managesieve protocol. I
>>> hope this is something that you would consider to add. Comments on the
>>> code are very welcome! Also, I hope I submitted this the right way. If
>>> I need to change anything, please let me know.
>>>
>>> Kind regards,
>>>
>>> Sander Hoentjen
>>>
>>> On 4/8/20 8:26 PM, Sander Hoentjen wrote:
>>>> # HG changeset patch
>>>> # User Sander Hoentjen <shoentjen@antagonist.nl>
>>>> # Date 1586369831 -7200
>>>> # Wed Apr 08 20:17:11 2020 +0200
>>>> # Node ID f1dffaf619688aaab90caf31781ebe27c3f79598
>>>> # Parent 0cb942c1c1aa98118076e72e0b89940e85e6291c
>>>> Added support for proxying managesieve protocol
>>>>
> [...]
>
>
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[PATCH] Added support for proxying managesieve protocol

Sander Hoentjen 441 April 08, 2020 02:28PM

[PATCH] Tests: added sieve tests

Sander Hoentjen 118 April 08, 2020 02:32PM

Re: [PATCH] Added support for proxying managesieve protocol

Sander Hoentjen 115 April 08, 2020 02:34PM

Re: [PATCH] Added support for proxying managesieve protocol

Sander Hoentjen 99 April 14, 2020 02:28PM

Re: [PATCH] Added support for proxying managesieve protocol

Maxim Konovalov 124 April 15, 2020 02:58PM

Re: [PATCH] Added support for proxying managesieve protocol

Sander Hoentjen 154 April 16, 2020 01:52PM

Re: [PATCH] Added support for proxying managesieve protocol

Sander Hoentjen 108 June 05, 2020 04:00AM

Re: [PATCH] Added support for proxying managesieve protocol

Sander Hoentjen 59 May 06, 2021 04:16PM

Re: [PATCH] Added support for proxying managesieve protocol

Sander Hoentjen 17 October 19, 2021 04:50AM

[PATCH] Added support for proxying managesieve protocol (fixed format)

Sander Hoentjen 114 April 08, 2020 02:40PM

[PATCH] Tests: added sieve tests (fixed format)

Sander Hoentjen 117 April 08, 2020 02:40PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 69
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready