Build: #1780 was successful Scheduled with changes by 4 people

Stages & jobs

  1. Build and Package

  2. Copy to Website

Build result summary

Details

Completed
Queue duration
23 minutes
Duration
40 minutes
Labels
None
Revision
8c8009e66f28f3f7c6cc0c76e28651eb3258ba56
Total tests
1342
First to pass since
#1779 (Scheduled – )

Tests

Code commits

Author Commit Message Commit date
daryl herzmann <akrherz@iastate.edu> daryl herzmann <akrherz@iastate.edu> 8c8009e66f28f3f7c6cc0c76e28651eb3258ba56 Merge pull request #2254 from igniterealtime/dependabot/github_actions/actions/checkout-4
Bump actions/checkout from 3 to 4
daryl herzmann <akrherz@iastate.edu> daryl herzmann <akrherz@iastate.edu> 37be33b781e815e935b6478e899766da2d20fa24 Merge pull request #2253 from guusdk/OF-2648_Fix-stream-namespace-restoration
OF-2648: Restore restoration of stream namespaces
Guus der Kinderen Guus der Kinderen 1bb4789ae22246a28255b20c42f9550b31ed58e6 OF-2648: Restore restoration of stream namespaces
In OF-2556, a workaround was provided to 'restore' namespace declarations on a stream element that had been parsed during the setup of the session.

This workaround injected a prefix-less 'stream' element. Unless the original stream defined a default namespace, this will not properly be parsed by an XML parser.

In this commit, the prefix 'stream' is used for the injected element (which is a de-facto standard in XMPP), and, for good measure, the namespace for that prefix is added, if it wasn't already recorded before.
dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> 9986ee789444deb4d0535f5a538d5e4ff62b4042 Bump actions/checkout from 3 to 4
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Guus der Kinderen Guus der Kinderen 534d837d31b7df181b59e916412e921b281f8820 OF-2647: Restore JiveGlobals.getHomeDirectory(String)
A previous commit for OF-2647 replaced the argument to the getter and setter of `JiveGlobals.setHomeDirectory()` from a String to a Path.

This leads to compatibility issues in various plugins:
- https://github.com/igniterealtime/openfire-bookmarks-plugin/issues/20
- https://github.com/igniterealtime/openfire-certificateManager-plugin/issues/19
- https://github.com/igniterealtime/openfire-monitoring-plugin/issues/362

In hindsight, the signature change was to much of a change. This commit 'undoes' that change, by:
- introducing `get/setHomePath(Path)` (instead of replacing `getHomeDirectory`)
- restoring `get/setHomeDirectory(String)` and marking it as being deprecated

This should restore backwards compatilibity, and give plugin implementors time to migrate from the old to the new API.

Tests

Fixed tests 1
Status Test Failing since View job Duration
Successful LocalIncomingServerSessionTest incomingTest(ServerSettings, ServerSettings)[255]
Failing since build #1779 (Scheduled) Debian Workflow < 1 sec

Jira issues

IssueDescriptionStatus
Unknown Issue TypeOF-2556Could not obtain issue details from Jira
Unknown Issue TypeOF-2647Could not obtain issue details from Jira
Unknown Issue TypeOF-2648Could not obtain issue details from Jira
Unknown Issue TypeOF-2654Could not obtain issue details from Jira
Unknown Issue TypeOF-2655Could not obtain issue details from Jira