Skip to content

Release/auditor 10.9 revise Oracle Database configuration and requirements#1000

Draft
pavelshabanov2025 wants to merge 10 commits into
devfrom
release/auditor_10.9
Draft

Release/auditor 10.9 revise Oracle Database configuration and requirements#1000
pavelshabanov2025 wants to merge 10 commits into
devfrom
release/auditor_10.9

Conversation

@pavelshabanov2025

Copy link
Copy Markdown
Contributor

Summary

  • Rewrote permissions.md: restructured into Prerequisites, numbered steps, DBA Role section, and Required Object Privileges table; removed "Step N" format and 12c references
  • Fixed database.md Step 3: restored HTML table formatting for Unified Auditing commands
  • Updated overview.md: removed automatic configuration option, removed duplicate CAUTION block, removed 11g/12c remnants
  • Fixed verifysettings.md: split Oracle 19c row to reflect column rename in 19c (ENTITY_NAME/ENABLED_OPTION vs USER_NAME/ENABLED_OPT for 12.2/18c); removed Oracle 11g row
  • Deleted unified.md: irrelevant for 10.9 since minimum supported Oracle version is 12.2, which has Unified Auditing natively
  • Updated software.md and supporteddatasources.md: removed Oracle 11g and 12c references, replaced with 12.2

Generated with AI

Co-Authored-By: Claude Code ai@netwrix.com

dsalyamova and others added 5 commits June 3, 2026 11:40
Two new pages covering State-in-Time report capabilities and the
configuration steps for enabling State-in-Time data collection in
Azure Files monitoring plans.
…0.9 [US 439213] (#1083)

* docs(auditor): add Group.Read.All API permission to AFA Azure Files 10.9 [US 439213]

Port Group.Read.All permission requirement from Auditor 10.8 (commit 2656597)
to the 10.9 documentation. Required by the AFA collector to resolve groups and
search by SID from DACLs, verified by live SIT testing.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>

* docs(auditor): apply 4 style fixes from PR #1047 review comments

- Remove "used in the points above" from opening Note (wordy phrase)
- Fix passive voice + missing article in Prerequisites ("is configured for data storage account" → "configured for the data storage account")
- Remove passive "You should" from Azure App Registration intro, add trailing period
- Remove redundant "find and" from IAM Roles Step 1 search instruction

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>

---------

Co-authored-by: Claude Code <ai@netwrix.com>
…y-pick from #1021) (#1088)

Co-authored-by: Claude (claude-sonnet-4-6) <noreply@anthropic.com>
…k from #1022) (#1089)

Co-authored-by: Claude (claude-sonnet-4-6) <noreply@anthropic.com>
…herry-pick from #1026) (#1090)

Co-authored-by: Claude (claude-sonnet-4-6) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Code Review

Reviewed the non-content changes (src/config/products.js, sidebars/auditor/10.9.js). No code correctness, security, or build-config issues found.

Findings:

  • src/config/products.js — Correctly adds 10.9 as a new version, flips isLatest from 10.8 to 10.9, and bumps defaultVersion to 10.9. Pattern matches how prior version bumps were handled.
  • sidebars/auditor/10.9.js — Identical to sidebars/auditor/10.8.js (autogenerated from dirName: '.'). Path matches sidebarFile declared in products.js. Good.
  • No docusaurus.config.js changes needed — plugin instances are auto-generated from PRODUCTS, so the two edits above are sufficient to wire the new version into routing, navbar dropdowns, and builds.
  • docs/auditor/10.9/index.md exists, so the version root resolves.
  • No script, workflow, secret, or KB-pipeline changes.

Non-blocking note (not a code issue): The PR title/description describe a focused Oracle-configuration revision, but the diff actually introduces the entire new 10.9 product version (~950 files, +37k lines). Worth confirming this matches the intended scope before merge, since reviewers reading just the description may not realize a full version is being added.

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.

4 participants