Skip to content

Numerix Docs and Readme - #193

Merged
a0d00kc merged 2 commits into
developfrom
feat/numerix-readme
Sep 27, 2025
Merged

Numerix Docs and Readme#193
a0d00kc merged 2 commits into
developfrom
feat/numerix-readme

Conversation

@Adit2607

Copy link
Copy Markdown
Contributor

🔁 Pull Request Template – BharatMLStack

Please fill out the following sections to help us review your changes efficiently.


📌 Summary

e.g., Adds optimizes Redis fetch latency in online-feature-store, or improves search UI responsiveness in trufflebox-ui.


📂 Modules Affected

  • horizon (Real-time systems / networking)
  • online-feature-store (Feature serving infra)
  • trufflebox-ui (Admin panel / UI)
  • infra (Docker, CI/CD, GCP/AWS setup)
  • docs (Documentation updates)
  • Other: ___________

✅ Type of Change

  • Feature addition
  • Bug fix
  • Infra / build system change
  • Performance improvement
  • Refactor
  • Documentation
  • Other: ___________

📊 Benchmark / Metrics (if applicable)

@Adit2607
Adit2607 changed the base branch from main to develop September 16, 2025 13:05
@semgrep-sast-meesho

Copy link
Copy Markdown

Semgrep found 1 grpc-client-new-insecure-connection finding:

Found an insecure gRPC connection using 'insecure.NewCredentials()'. This creates a connection without encryption to a gRPC server. A malicious attacker could tamper with the gRPC message, which could compromise the machine. Instead, establish a secure connection with an SSL certificate using the 'grpc.WithTransportCredentials()' function. You can create a create credentials using a 'tls.Config{}' struct with 'credentials.NewTLS()'. The final fix looks like this: 'grpc.WithTransportCredentials(credentials.NewTLS())'.

View Dataflow Graph
flowchart LR
    classDef invis fill:white, stroke: none
    classDef default fill:#e7f5ff, color:#1c7fd6, stroke: none

    subgraph File0["<b>horizon/pkg/grpc/grpc.go</b>"]
        direction LR
        %% Source

        subgraph Source
            direction LR

            v0["<a href=https://github.com/Meesho/BharatMLStack/blob/1c4db2fe693acf83c959a0ee0993e32c40efaf6a/horizon/pkg/grpc/grpc.go#L14 target=_blank style='text-decoration:none; color:#1c7fd6'>[Line: 14] insecure.NewCredentials()</a>"]
        end
        %% Intermediate

        %% Sink

        subgraph Sink
            direction LR

            v1["<a href=https://github.com/Meesho/BharatMLStack/blob/1c4db2fe693acf83c959a0ee0993e32c40efaf6a/horizon/pkg/grpc/grpc.go#L14 target=_blank style='text-decoration:none; color:#1c7fd6'>[Line: 14] insecure.NewCredentials()</a>"]
        end
    end
    %% Class Assignment
    Source:::invis
    Sink:::invis

    File0:::invis

    %% Connections

    Source --> Sink

Loading

@a0d00kc
a0d00kc merged commit 0c293de into develop Sep 27, 2025
8 checks passed
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