Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 30, 2024

Bumps the npm_and_yarn group with 1 update in the /scripts/node_modules.sh/debug directory: karma.

Updates karma from 3.1.4 to 6.4.4

Release notes

Sourced from karma's releases.

v6.4.4

6.4.4 (2024-07-29)

v6.4.3

6.4.3 (2024-02-24)

Bug Fixes

  • add build commits for patch release (d7f2d69)

v6.4.2

6.4.2 (2023-04-21)

Bug Fixes

v6.4.1

6.4.1 (2022-09-19)

Bug Fixes

v6.4.0

6.4.0 (2022-06-14)

Features

  • support SRI verification of link tags (dc51a2e)
  • support SRI verification of script tags (6a54b1c)

v6.3.20

6.3.20 (2022-05-13)

Bug Fixes

  • prefer IPv4 addresses when resolving domains (e17698f), closes #3730

v6.3.19

6.3.19 (2022-04-19)

Bug Fixes

... (truncated)

Changelog

Sourced from karma's changelog.

6.4.4 (2024-07-29)

6.4.3 (2024-02-24)

Bug Fixes

  • add build commits for patch release (d7f2d69)

6.4.2 (2023-04-21)

Bug Fixes

6.4.1 (2022-09-19)

Bug Fixes

6.4.0 (2022-06-14)

Features

  • support SRI verification of link tags (dc51a2e)
  • support SRI verification of script tags (6a54b1c)

6.3.20 (2022-05-13)

Bug Fixes

  • prefer IPv4 addresses when resolving domains (e17698f), closes #3730

6.3.19 (2022-04-19)

Bug Fixes

  • client: error out when opening a new tab fails (099b85e)

6.3.18 (2022-04-13)

Bug Fixes

... (truncated)

Commits
  • 84f85e7 chore(release): 6.4.4 [skip ci]
  • a4d1284 build(deps-dev): bump ws from 6.2.1 to 6.2.3
  • d8cf806 chore(release): 6.4.3 [skip ci]
  • d7f2d69 fix: add build commits for patch release
  • 85a2eeb build(deps-dev): bump decode-uri-component from 0.2.0 to 0.2.2
  • 0bffce2 build(deps): updated socket.io version to fix security issues with socket.io-...
  • 86667ab build(deps): bump follow-redirects from 1.11.0 to 1.15.4
  • 450fdfd docs: Add deprecation notice to Karma README
  • 9de3c00 chore(release): 6.4.2 [skip ci]
  • c6a4271 fix: few typos
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions
    You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps the npm_and_yarn group with 1 update in the /scripts/node_modules.sh/debug directory: [karma](https://github.com/karma-runner/karma).


Updates `karma` from 3.1.4 to 6.4.4
- [Release notes](https://github.com/karma-runner/karma/releases)
- [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md)
- [Commits](karma-runner/karma@v3.1.4...v6.4.4)

---
updated-dependencies:
- dependency-name: karma
  dependency-type: direct:development
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Aug 30, 2024
@CodeMonkeyCybersecurity
Copy link
Owner

Updates karma from 3.1.4 to 6.4.4

@CodeMonkeyCybersecurity
Copy link
Owner

Bump karma from 3.1.4 to 6.4.4 in /scripts/node_modules.sh/debug in the npm_and_yarn group across 1 directory #1

CodeMonkeyCybersecurity added a commit that referenced this pull request Oct 13, 2025
…ake LiteLLM default

CRITICAL FIXES - DO NOT DEPLOY WITHOUT THESE:

Issue #1: Open WebUI Not Connected to LiteLLM (CRITICAL)
- Added OPENAI_API_BASE_URL=http://litellm-proxy:4000 to docker-compose.yml
- Added OPENAI_API_KEY=${LITELLM_MASTER_KEY} to docker-compose.yml
- Added connection variables to .env template
- Without this, Open WebUI has NO backend and is completely non-functional

Issue #2: LiteLLM Now DEFAULT (as requested)
- Changed from --use-litellm flag to --direct-mode flag
- LiteLLM production mode is now enabled by default
- Direct mode is opt-in for development only
- Updated all help text and examples

Issue #3: Enhanced User Experience
- Clear indication of production vs development mode
- Warning message when using direct mode
- Production features list shown by default

TECHNICAL DETAILS:

Docker Compose Changes:
- openwebui service now has environment block
- OPENAI_API_BASE_URL uses Docker service name (litellm-proxy:4000)
- OPENAI_API_KEY references LITELLM_MASTER_KEY from .env
- Proper service-to-service communication on Docker network

Environment File Changes:
- Added OPENAI_API_BASE_URL=http://litellm-proxy:4000
- Added OPENAI_API_KEY=${LITELLM_MASTER_KEY}
- These tell Open WebUI where to find LiteLLM

CLI Changes:
- Removed: --use-litellm flag
- Added: --direct-mode flag (disables LiteLLM)
- Default behavior: LiteLLM enabled
- Updated examples to show default is production-ready

Type Changes:
- Added DirectMode bool field
- UseLiteLLM set to true by default when DirectMode is false
- Clear separation between production and development modes

Success Message Changes:
- Shows LiteLLM URLs by default
- Lists production features enabled
- Warning when direct mode is used

VERIFICATION:
✅ AZURE_MODEL format correct: azure/gpt-4
✅ Docker service names correct: litellm-proxy
✅ LiteLLM config structure correct
✅ Volume mounts correct
✅ Open WebUI connection configured
✅ LiteLLM is default mode
✅ All packages compile successfully

This fixes the critical issue where Open WebUI would deploy but have
no backend to connect to, resulting in a completely non-functional system.

Reference: OPENWEBUI_LITELLM_REVIEW.md for detailed technical review
CodeMonkeyCybersecurity added a commit that referenced this pull request Oct 13, 2025
…ment

VERIFICATION COMPLETE - ALL ISSUES FIXED ✅

Confirmed in current codebase:
✅ Issue #1: Open WebUI connection to LiteLLM - FIXED (lines 727-732)
✅ Issue #2: LiteLLM default mode - FIXED (lines 95-97, cmd flags)
✅ Issue #3: Environment variables - FIXED (lines 612-614)

All critical configuration points verified:
✅ AZURE_MODEL format: azure/%s (line 598)
✅ Service name: http://litellm-proxy:4000 (not localhost)
✅ LiteLLM config: os.environ/AZURE_MODEL (line 673)
✅ Volume mount: ./litellm_config.yaml:/app/config.yaml (line 737)
✅ DirectMode defaults to false (LiteLLM enabled by default)

Status: READY FOR TESTING

Includes complete testing checklist and deployment steps.
CodeMonkeyCybersecurity pushed a commit that referenced this pull request Nov 22, 2025
Architecture enforcement: Extracted ALL business logic from cmd/debug/iris.go
to pkg/iris/debug/ following Assess → Intervene → Evaluate pattern.

Changes:
- Reduce cmd/debug/iris.go from 1507 lines → 65 lines (96% reduction)
- Create pkg/iris/debug/ package with 7 files:
  * types.go (45 lines) - IrisConfig, CheckResult, DiagnosticConfig
  * diagnostics.go (101 lines) - Main RunDiagnostics() orchestration
  * checks_infrastructure.go (447 lines) - Project, Temporal, ports, binaries
  * checks_config.go (172 lines) - Config, Azure OpenAI, SMTP
  * checks_services.go (355 lines) - Worker, webhook, systemd, workflows
  * checks_system.go (71 lines) - Go dependencies
  * utils.go (391 lines) - Display, binary location, test alerts

Architecture Compliance:
- cmd/debug/iris.go now pure orchestration (flag parsing + delegation)
- All business logic in pkg/ with proper error handling
- Follows Assess → Intervene → Evaluate pattern
- All exported functions have godoc comments
- Structured logging with otelzap.Ctx(rc.Ctx)
- No file operations, loops, or complex conditionals in cmd/

Verification:
- go build ./pkg/iris/debug/... passes ✓
- go vet ./pkg/iris/debug/... passes ✓
- Zero functionality lost
- 25 functions successfully migrated

Impact:
- cmd/debug/iris.go: 1507 → 65 lines (1442 lines extracted)
- Improved testability (pkg/ functions are unit-testable)
- Better code organization (checks grouped by category)
- Maintains all diagnostic functionality

Part of systematic cmd/ to pkg/ migration (#1 of 15 files).
Next: cmd/debug/bootstrap.go (853 lines)
@CodeMonkeyCybersecurity CodeMonkeyCybersecurity merged commit bb762cc into main Nov 25, 2025
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/scripts/node_modules.sh/debug/npm_and_yarn-1dc217bc5b branch November 25, 2025 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants