Skip to content

docs(ospo): address review feedback on community health rollout - #432

Merged
DeepDiver1975 merged 1 commit into
owncloud:ospo/community-health-v2from
dj4oC:fix/review-feedback-425
Jul 31, 2026
Merged

docs(ospo): address review feedback on community health rollout#432
DeepDiver1975 merged 1 commit into
owncloud:ospo/community-health-v2from
dj4oC:fix/review-feedback-425

Conversation

@dj4oC

@dj4oC dj4oC commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Addresses phil-davis's review comments on #425:

  • agents.md: note tests/ covers acceptance tests too, and list the PHPUnit/API/CLI/WebUI acceptance make targets individually (as suggested)
  • README.md: drop the outdated "encryption cannot be disabled" claim — verified occ encryption:decrypt-all (core) explicitly disables encryption and decrypts all files, followed by occ encryption:disable — and restore OpenSSL 3.x legacy-cipher guidance that had been dropped
  • README.md: stop recommending per-user key encryption — verified it's marked deprecated in templates/settings-admin.php, linking to the server release notes

Base is ospo/community-health-v2 (not master) so merging this lands the fixes directly on #425.

🤖 Generated with Claude Code

- agents.md: note tests/ covers acceptance tests too, and list the
  PHPUnit/API/CLI/WebUI acceptance make targets individually
  (per phil-davis review comments)
- README: drop the outdated "encryption cannot be disabled" claim —
  `occ encryption:decrypt-all` + `occ encryption:disable` reverse it —
  and restore the OpenSSL 3.x legacy-cipher guidance that was dropped
  (per phil-davis review comment)
- README: stop recommending per-user key encryption, which is
  deprecated per the server release notes (per phil-davis review
  comment)

🤖 Generated with Claude Code

Signed-off-by: David Walter <david.walter@kiteworks.com>
@DeepDiver1975
DeepDiver1975 merged commit 8b926c9 into owncloud:ospo/community-health-v2 Jul 31, 2026
11 checks passed
phil-davis pushed a commit that referenced this pull request Aug 1, 2026
…iles (#425)

* docs(ospo): community health rollout v2 — README, agents.md, health files

Introduced by the Kiteworks Open Source Program Office (OSPO) on May 5, 2026.

Changes:
- README.md: rewritten with OSPO v2 template — license-specific migration
  guidance, Community & Support section, Contributing workflow, Security
  section pointing to security.owncloud.com + YesWeHack bug bounty
- agents.md: AI agent context file with architecture, build commands, and
  OSPO Policy Constraints (GitHub Actions, Dependabot, Git Workflow)
- CODE_OF_CONDUCT.md: redirect to https://owncloud.com/contribute/code-of-conduct/
- CONTRIBUTING.md: redirect to https://owncloud.com/contribute/
- SECURITY.md: redirect to https://security.owncloud.com + YesWeHack
- SUPPORT.md: redirect to https://owncloud.com/contact-us/ + channels

OSPO: https://kiteworks.com/opensource

Signed-off-by: David Walter <david.walter@kiteworks.com>

* docs(ospo): address review feedback on community health rollout (#432)

- agents.md: note tests/ covers acceptance tests too, and list the
  PHPUnit/API/CLI/WebUI acceptance make targets individually
  (per phil-davis review comments)
- README: drop the outdated "encryption cannot be disabled" claim —
  `occ encryption:decrypt-all` + `occ encryption:disable` reverse it —
  and restore the OpenSSL 3.x legacy-cipher guidance that was dropped
  (per phil-davis review comment)
- README: stop recommending per-user key encryption, which is
  deprecated per the server release notes (per phil-davis review
  comment)

🤖 Generated with Claude Code

Signed-off-by: David Walter <david.walter@kiteworks.com>

---------

Signed-off-by: David Walter <david.walter@kiteworks.com>
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

Successfully merging this pull request may close these issues.

2 participants