Re: [VOTE] [log4net] Release log4net 2.0.10
My +1 Dominic, if you are going to vote you need to do it formally. Ralph > On Sep 10, 2020, at 12:57 AM, Dominik Psenner wrote: > > Knowing that those changes are intentional I am confident that the next > release is better than the last. This is reason enough to move on. If > something breaks we can still address those issues with another future > release. > -- > Sent from my phone. Typos are a kind gift to anyone who happens to find > them. > >> On Thu, Sep 10, 2020, 09:20 Davyd McColl wrote: >> >> Hi Dominik >> >> I have had a long look over the changes (both via the PR and locally, as I >> contributed to help with some infra changes) and I'm happy -- there's been >> a lot of clean-up and simplification and in addition, tests are now run >> against all targets -- so that's a good thing. Some of these changes are >> required to resolve issues for netstandard2.0 users who have upgraded to >> 2.0.9. Community member NicholasNoise put in a lot of work on this. >> >> If it helps, the original PR is here: >> https://github.com/apache/logging-log4net/pull/63 -- it makes viewing >> changes a lot simpler. Changes to a lot of the #ifdefs are updates from >> NETSTANDARD1_3 to simply NETSTANDARD as many of the changes required for >> netstandard2.0 are compatible. I contributed on that PR too, mainly around >> getting build to work as expected. >> >> You're welcome to use the npm-based build / test pipeline to verify: I've >> just updated master to automatically test across all platforms when running >> `npm test`, so it should be easy to verify that all things are functional: >> - install node if you don't have it yet (I suggest via nvm) >> - `npm ci` >> - `npm test` >> >> (assuming that you have all the required build targets -- there are helper >> .ps1 scripts to get the older targets -- netcore 1.1 and netfx35) >> >> -d >> >> >> On 2020/09/10 09:03:45, Dominik Psenner wrote: >> Hi >> >> Sorry to not have responded earlier. Time is short and the days are busy. I >> looked at the diff and found several suspicious changes. Several hundred >> ifdefs have been removed/replaced along with tests. Therefore I have a bad >> feeling about those changes without further careful checking. I propose to >> release the cve fix alone and follow up a second release as soon as someone >> had the time to verify that the netstandard2 changes are ok. >> >> Best regards >> -- >> Sent from my phone. Typos are a kind gift to anyone who happens to find >> them. >> >>> On Thu, Sep 10, 2020, 08:48 Davyd McColl wrote: >>> >>> Hi >>> >>> Sorry to be a bother, but I haven't heard anything back on this apart >> from >>> Dominik's inquiry into netstandard 1.3 support. I'd really like to get >> this >>> out as: >>> a) it contains the CVE fix that has been asked about so much >>> b) it solves some issues affecting netstandard users >>> >>> Thanks >>> -d >>> On 2020/09/06 20:51:38, Davyd McColl wrote: >>> Hi all >>> >>> I'd like to propose a vote to release 2.0.10 of log4net, with: >>> - updated netstandard 2.0 support from community member NicholasNoise >>> - cherry-picked fix for CVE-2018-1285 (I had to modify slightly since the >>> mechanism used there is outdated for netstandard 2.0, but the principle >>> stands >>> >>> I've created an RC release at GitHub: >>> https://github.com/apache/logging-log4net/releases/tag/v2.0.10-rc1 and >>> pushed updated site material to the `asf-staging` branch of the >>> logging-log4net-site repo. >>> >>> Thanks >>> -d >>
Re: [VOTE] [log4net] Release log4net 2.0.10
I'll have time to review the release over the weekend. I'm going on PTO starting this weekend, too, so I'll likely be doing stuff here a few random days over the next couple weeks, too. On Thu, 10 Sep 2020 at 02:57, Dominik Psenner wrote: > > Knowing that those changes are intentional I am confident that the next > release is better than the last. This is reason enough to move on. If > something breaks we can still address those issues with another future > release. > -- > Sent from my phone. Typos are a kind gift to anyone who happens to find > them. > > On Thu, Sep 10, 2020, 09:20 Davyd McColl wrote: > > > Hi Dominik > > > > I have had a long look over the changes (both via the PR and locally, as I > > contributed to help with some infra changes) and I'm happy -- there's been > > a lot of clean-up and simplification and in addition, tests are now run > > against all targets -- so that's a good thing. Some of these changes are > > required to resolve issues for netstandard2.0 users who have upgraded to > > 2.0.9. Community member NicholasNoise put in a lot of work on this. > > > > If it helps, the original PR is here: > > https://github.com/apache/logging-log4net/pull/63 -- it makes viewing > > changes a lot simpler. Changes to a lot of the #ifdefs are updates from > > NETSTANDARD1_3 to simply NETSTANDARD as many of the changes required for > > netstandard2.0 are compatible. I contributed on that PR too, mainly around > > getting build to work as expected. > > > > You're welcome to use the npm-based build / test pipeline to verify: I've > > just updated master to automatically test across all platforms when running > > `npm test`, so it should be easy to verify that all things are functional: > > - install node if you don't have it yet (I suggest via nvm) > > - `npm ci` > > - `npm test` > > > > (assuming that you have all the required build targets -- there are helper > > .ps1 scripts to get the older targets -- netcore 1.1 and netfx35) > > > > -d > > > > > > On 2020/09/10 09:03:45, Dominik Psenner wrote: > > Hi > > > > Sorry to not have responded earlier. Time is short and the days are busy. I > > looked at the diff and found several suspicious changes. Several hundred > > ifdefs have been removed/replaced along with tests. Therefore I have a bad > > feeling about those changes without further careful checking. I propose to > > release the cve fix alone and follow up a second release as soon as someone > > had the time to verify that the netstandard2 changes are ok. > > > > Best regards > > -- > > Sent from my phone. Typos are a kind gift to anyone who happens to find > > them. > > > > On Thu, Sep 10, 2020, 08:48 Davyd McColl wrote: > > > > > Hi > > > > > > Sorry to be a bother, but I haven't heard anything back on this apart > > from > > > Dominik's inquiry into netstandard 1.3 support. I'd really like to get > > this > > > out as: > > > a) it contains the CVE fix that has been asked about so much > > > b) it solves some issues affecting netstandard users > > > > > > Thanks > > > -d > > > > > > On 2020/09/06 20:51:38, Davyd McColl wrote: > > > Hi all > > > > > > I'd like to propose a vote to release 2.0.10 of log4net, with: > > > - updated netstandard 2.0 support from community member NicholasNoise > > > - cherry-picked fix for CVE-2018-1285 (I had to modify slightly since the > > > mechanism used there is outdated for netstandard 2.0, but the principle > > > stands > > > > > > I've created an RC release at GitHub: > > > https://github.com/apache/logging-log4net/releases/tag/v2.0.10-rc1 and > > > pushed updated site material to the `asf-staging` branch of the > > > logging-log4net-site repo. > > > > > > Thanks > > > -d > > -- Matt Sicker
Re: [VOTE] [log4net] Release log4net 2.0.10
Knowing that those changes are intentional I am confident that the next release is better than the last. This is reason enough to move on. If something breaks we can still address those issues with another future release. -- Sent from my phone. Typos are a kind gift to anyone who happens to find them. On Thu, Sep 10, 2020, 09:20 Davyd McColl wrote: > Hi Dominik > > I have had a long look over the changes (both via the PR and locally, as I > contributed to help with some infra changes) and I'm happy -- there's been > a lot of clean-up and simplification and in addition, tests are now run > against all targets -- so that's a good thing. Some of these changes are > required to resolve issues for netstandard2.0 users who have upgraded to > 2.0.9. Community member NicholasNoise put in a lot of work on this. > > If it helps, the original PR is here: > https://github.com/apache/logging-log4net/pull/63 -- it makes viewing > changes a lot simpler. Changes to a lot of the #ifdefs are updates from > NETSTANDARD1_3 to simply NETSTANDARD as many of the changes required for > netstandard2.0 are compatible. I contributed on that PR too, mainly around > getting build to work as expected. > > You're welcome to use the npm-based build / test pipeline to verify: I've > just updated master to automatically test across all platforms when running > `npm test`, so it should be easy to verify that all things are functional: > - install node if you don't have it yet (I suggest via nvm) > - `npm ci` > - `npm test` > > (assuming that you have all the required build targets -- there are helper > .ps1 scripts to get the older targets -- netcore 1.1 and netfx35) > > -d > > > On 2020/09/10 09:03:45, Dominik Psenner wrote: > Hi > > Sorry to not have responded earlier. Time is short and the days are busy. I > looked at the diff and found several suspicious changes. Several hundred > ifdefs have been removed/replaced along with tests. Therefore I have a bad > feeling about those changes without further careful checking. I propose to > release the cve fix alone and follow up a second release as soon as someone > had the time to verify that the netstandard2 changes are ok. > > Best regards > -- > Sent from my phone. Typos are a kind gift to anyone who happens to find > them. > > On Thu, Sep 10, 2020, 08:48 Davyd McColl wrote: > > > Hi > > > > Sorry to be a bother, but I haven't heard anything back on this apart > from > > Dominik's inquiry into netstandard 1.3 support. I'd really like to get > this > > out as: > > a) it contains the CVE fix that has been asked about so much > > b) it solves some issues affecting netstandard users > > > > Thanks > > -d > > > > On 2020/09/06 20:51:38, Davyd McColl wrote: > > Hi all > > > > I'd like to propose a vote to release 2.0.10 of log4net, with: > > - updated netstandard 2.0 support from community member NicholasNoise > > - cherry-picked fix for CVE-2018-1285 (I had to modify slightly since the > > mechanism used there is outdated for netstandard 2.0, but the principle > > stands > > > > I've created an RC release at GitHub: > > https://github.com/apache/logging-log4net/releases/tag/v2.0.10-rc1 and > > pushed updated site material to the `asf-staging` branch of the > > logging-log4net-site repo. > > > > Thanks > > -d >
Re: [VOTE] [log4net] Release log4net 2.0.10
Hi Dominik I have had a long look over the changes (both via the PR and locally, as I contributed to help with some infra changes) and I'm happy -- there's been a lot of clean-up and simplification and in addition, tests are now run against all targets -- so that's a good thing. Some of these changes are required to resolve issues for netstandard2.0 users who have upgraded to 2.0.9. Community member NicholasNoise put in a lot of work on this. If it helps, the original PR is here: https://github.com/apache/logging-log4net/pull/63 -- it makes viewing changes a lot simpler. Changes to a lot of the #ifdefs are updates from NETSTANDARD1_3 to simply NETSTANDARD as many of the changes required for netstandard2.0 are compatible. I contributed on that PR too, mainly around getting build to work as expected. You're welcome to use the npm-based build / test pipeline to verify: I've just updated master to automatically test across all platforms when running `npm test`, so it should be easy to verify that all things are functional: - install node if you don't have it yet (I suggest via nvm) - `npm ci` - `npm test` (assuming that you have all the required build targets -- there are helper .ps1 scripts to get the older targets -- netcore 1.1 and netfx35) -d On 2020/09/10 09:03:45, Dominik Psenner wrote: Hi Sorry to not have responded earlier. Time is short and the days are busy. I looked at the diff and found several suspicious changes. Several hundred ifdefs have been removed/replaced along with tests. Therefore I have a bad feeling about those changes without further careful checking. I propose to release the cve fix alone and follow up a second release as soon as someone had the time to verify that the netstandard2 changes are ok. Best regards -- Sent from my phone. Typos are a kind gift to anyone who happens to find them. On Thu, Sep 10, 2020, 08:48 Davyd McColl wrote: > Hi > > Sorry to be a bother, but I haven't heard anything back on this apart from > Dominik's inquiry into netstandard 1.3 support. I'd really like to get this > out as: > a) it contains the CVE fix that has been asked about so much > b) it solves some issues affecting netstandard users > > Thanks > -d > > On 2020/09/06 20:51:38, Davyd McColl wrote: > Hi all > > I'd like to propose a vote to release 2.0.10 of log4net, with: > - updated netstandard 2.0 support from community member NicholasNoise > - cherry-picked fix for CVE-2018-1285 (I had to modify slightly since the > mechanism used there is outdated for netstandard 2.0, but the principle > stands > > I've created an RC release at GitHub: > https://github.com/apache/logging-log4net/releases/tag/v2.0.10-rc1 and > pushed updated site material to the `asf-staging` branch of the > logging-log4net-site repo. > > Thanks > -d
Re: [VOTE] [log4net] Release log4net 2.0.10
Hi Sorry to not have responded earlier. Time is short and the days are busy. I looked at the diff and found several suspicious changes. Several hundred ifdefs have been removed/replaced along with tests. Therefore I have a bad feeling about those changes without further careful checking. I propose to release the cve fix alone and follow up a second release as soon as someone had the time to verify that the netstandard2 changes are ok. Best regards -- Sent from my phone. Typos are a kind gift to anyone who happens to find them. On Thu, Sep 10, 2020, 08:48 Davyd McColl wrote: > Hi > > Sorry to be a bother, but I haven't heard anything back on this apart from > Dominik's inquiry into netstandard 1.3 support. I'd really like to get this > out as: > a) it contains the CVE fix that has been asked about so much > b) it solves some issues affecting netstandard users > > Thanks > -d > > On 2020/09/06 20:51:38, Davyd McColl wrote: > Hi all > > I'd like to propose a vote to release 2.0.10 of log4net, with: > - updated netstandard 2.0 support from community member NicholasNoise > - cherry-picked fix for CVE-2018-1285 (I had to modify slightly since the > mechanism used there is outdated for netstandard 2.0, but the principle > stands > > I've created an RC release at GitHub: > https://github.com/apache/logging-log4net/releases/tag/v2.0.10-rc1 and > pushed updated site material to the `asf-staging` branch of the > logging-log4net-site repo. > > Thanks > -d
Re: [VOTE] [log4net] Release log4net 2.0.10
Hi Sorry to be a bother, but I haven't heard anything back on this apart from Dominik's inquiry into netstandard 1.3 support. I'd really like to get this out as: a) it contains the CVE fix that has been asked about so much b) it solves some issues affecting netstandard users Thanks -d On 2020/09/06 20:51:38, Davyd McColl wrote: Hi all I'd like to propose a vote to release 2.0.10 of log4net, with: - updated netstandard 2.0 support from community member NicholasNoise - cherry-picked fix for CVE-2018-1285 (I had to modify slightly since the mechanism used there is outdated for netstandard 2.0, but the principle stands I've created an RC release at GitHub: https://github.com/apache/logging-log4net/releases/tag/v2.0.10-rc1 and pushed updated site material to the `asf-staging` branch of the logging-log4net-site repo. Thanks -d