Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release openidconnect 2.1.0 #168

Closed
36 of 45 tasks
hurradieweltgehtunter opened this issue Jul 21, 2021 · 15 comments
Closed
36 of 45 tasks

Release openidconnect 2.1.0 #168

hurradieweltgehtunter opened this issue Jul 21, 2021 · 15 comments

Comments

@hurradieweltgehtunter
Copy link
Contributor

hurradieweltgehtunter commented Jul 21, 2021

Reason

#148 [Security] Bump phpseclib/phpseclib from 2.0.29 to 2.0.31
#150 [Enhancement] AutoProvisioning based on a Provisioning Claim
#151 [Enhancement] Autoprovisionining based on claim
#155 [Bugfix] Use random_bytes for uid and password generation
#167 [Enhancement] Add db as additional settings storage backend

Release is required (mainly because of #167 ) for use in ownCloud Online.

Product approval

  • Verify transifex commits are in master: If a folder https://github.com/owncloud//tree/master/l10n exists, then https://drone.owncloud.com/owncloud/ should receive nightly '[tx] updated from transifex` ...
  • Ping pmaier to check info.xml description and screenshot
  • Get approval on translations from pmaier - send the date of the last tx commit from drone to Patrick

QA

  • Create QA Milestone and set the description to $version
  • Move tickets from milestone development to qa (testplan and tickets mentioned in release ticket)
  • Create release branch with the format "release-$version"
  • Create "$version Testplan" -> [QA] 2.1.0 Testplan #179
  • copy new/reopened issues from the previous release ticket as 'Issues to review' below.
  • check if Non-normal Release Process applies, and adapt this checklist.

Documentation

  • Changes in the docs needed? If so create issue in owncloud/docs.

Marketing

Build

All actions to be done on the release branch from here:

  • Bump version in info.xml (no version suffix)
  • Bump "sonar.projectVersion=" in sonar-project.properties (no version suffix)
  • Check php version dependency in info.xml
  • Check minimum ownCloud version dependency in info.xml
  • Change CHANGELOG.md 'Unreleased' to '$version' (today's date)

Beta/RC

  • Ping product owner to create github tag for v${version}-rc1 on release branch (see https://confluence.owncloud.com/display/EN/Product+Owners+List)
  • Build and sign RC tarball from github tag v${version}-rc1 (see handbook for how to build)
  • Upload as Github release using changelog entries
  • Manual testing (changelog, regression, explorative):
    • If this is an enterprise app: Enabling the app must start the grace period
    • If this is the initial release of an app: Check that the minimum version constraint resolves correctly
    • Changelog testing
    • Test and document in testplan => Final "Go" from QA (set 'QA-ready' label) => else do another RC after fixing issues in the "release-$version" branch
      • server + web ui + desktop regression tests
      • android regression tests
      • ios regression tests

Final

  • Check that no documentation issues are pending
  • Rename last RC tarball to final
  • Create github tag for final v${version} on release branch
  • Upload as Github release using changelog entries
  • Ping #marketing channel to coordinate publishing

Publishing

  • Upload tarball to marketplace (see handbook in confluence for details)
  • Publish tarball for customers - old 10, old 10.0 and new portal (see handbook for details)
  • Send mail to release-coordination with link to this ticket and to the changelog
  • Copy email to rocket chat #updates channel

Post-release

@jnweiger
Copy link
Contributor

jnweiger commented Jul 22, 2021

Testplan draft: #132
Please ping me for additions, I'll make a nice testplan from that.

I assume this will be a slow release, multiple new setups are needed here.

@jnweiger jnweiger added this to the QA milestone Jul 22, 2021
@hurradieweltgehtunter
Copy link
Contributor Author

@jnweiger Testplan needs the following additions regarding #167
For other PR's we need to ask PR creators
@IljaN @T0mWz @DeepDiver1975 could you please give information about PR's #148 #150 #151 #155 ?

Add no settings at all

  • Enable app
  • Test login

Expected behaviour

App fails the same way as previous release

Add no settings to DB

  • Add valid settings to config.php (default way, not altered)
  • Remove any existing config settings in DB
  • Enable app
  • Test login

Expected behaviour

Login should work

Add invalid settings to DB

Expected behaviour

  • Login should work
  • ownCloud.log should contain an error message, see
    'Loaded config from DB is not valid (malformed JSON); JSON Last Error: ' . json_last_error(),

Add valid settings to DB

Expected behaviour

Login should work

@jnweiger
Copy link
Contributor

@hurradieweltgehtunter can you provide test instances fro the above issues?

@hurradieweltgehtunter
Copy link
Contributor Author

I don't have access to set up public instances, Sorry.

@hurradieweltgehtunter
Copy link
Contributor Author

Is this going forward? Anything I can do to speed this up?

@michaelstingl
Copy link

Is this going forward? Anything I can do to speed this up?

No progress. Blocked by other priorities and vacations.

@jnweiger
Copy link
Contributor

@IljaN @T0mWz @DeepDiver1975 could you please give information about PR's #148 #150 #151 #155 ?

Do we have any details about these issues?

@DeepDiver1975
Copy link
Member

#148

library update

#150 #151

customer feature for surf

#155

library update

@DeepDiver1975
Copy link
Member

Also an important change in this release which needs to go out - #130

@T0mWz
Copy link
Contributor

T0mWz commented Sep 10, 2021

@jnweiger what information is required for #150 #151 ?
had described this little bit in issue #149, but maybe it's to short or more info required. Please let me know, you can also find me on Rocket Chat. 😃

@jnweiger jnweiger pinned this issue Sep 14, 2021
@jnweiger
Copy link
Contributor

@DeepDiver1975
Copy link
Member

Most recent tests with openidconnect app (git master) have shown that PKCE is not properly working.
Upstream fix is pending at the moment: jumbojett/OpenID-Connect-PHP#273

Affected IdPs: any who supports PKCE or enforces it.

@jnweiger
Copy link
Contributor

RC2 built. It includes the fix for #181

@jnweiger
Copy link
Contributor

jnweiger commented Nov 12, 2021

RC2 passed changelog regression tests.

@jnweiger jnweiger mentioned this issue Nov 17, 2021
@jnweiger jnweiger unpinned this issue Nov 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants