Skip to content

code-update-standardization #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 7 commits into from
Closed

Conversation

vlussenburg
Copy link
Collaborator

@vlussenburg vlussenburg commented May 10, 2025

✨ PR Description

Purpose: This PR implements order history tracking and improves error handling across multiple services.

Main changes:

  • Added date field to charge requests and responses in billing service
  • Implemented response history tracking in BillingController
  • Added error logging and ISO-8601 timestamp to OrderController in Java service

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We’d love your feedback! 🚀

Copy link

@gitstream-cm gitstream-cm bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR contains a TODO statement. Please check to see if they should be removed.

Copy link

gitstream-cm bot commented May 10, 2025

Please mark which AI tools you used for this PR by checking the appropriate boxes:

  • GitHub Copilot
  • Cursor
  • ChatGPT
  • Tabnine
  • JetBrains AI Assistant
  • VSCode IntelliCode
  • Claude
  • Gemini
  • Other AI tool
  • No AI tools were used

Tip: If you want to avoid this comment in the future, you can add a label of the format 🤖 ai-* when creating your PR.

Copy link

gitstream-cm bot commented May 10, 2025

✨ PR Review

General Feedback

The PR standardizes code across different services with some improvements like error logging and date tracking. However, there are several issues including a typo in the JSON payload field, a missing space in the Authorization header, and inconsistent properties between request and response in billing. Most changes appear to be related to timestamps, error handling improvements, and cleanup of auth users.

File: services/orders-java/src/main/java/com/example/orders/controller/OrderController.java
Bug - Field Name Mismatch

Details

Problem: Field Name Mismatch - There's a typo in the JSON field name "dats" in OrderController, but the corresponding field in BillingController is expecting "date". This will cause the timestamp functionality to break.
Fix: Change the field name from "dats" to "date" to match the expected field name in the BillingController.
Why: The inconsistent field name creates a data mismatch between services that will cause the timestamp data to be lost.

public class OrderController {
     JSONObject payload = new JSONObject();
     payload.put("username", username);
     payload.put("productId", productId);
     payload.put("quantity", quantity);
-    payload.put("dats", Instant.now().toString());
+    payload.put("date", Instant.now().toString());

File: frontend/public/app.js
Bug - Authorization Header Format

Details

Problem: Authorization Header Format - The Authorization header is missing a space between "Bearer" and the token, which will cause authentication failures as this violates the Bearer token format specification.
Fix: Add a space between "Bearer" and the token in the Authorization header.
Why: The incorrect format of the Authorization header will cause all API calls to fail authentication.

async function placeOrder() {
     method: "POST",
     headers: {
       "Content-Type": "application/json",
-      "Authorization": "Bearer" + token
+      "Authorization": "Bearer " + token
     },
     body: JSON.stringify({ productId, quantity }),
   });

File: services/auth-python/app.py
Security - Hardcoded Credentials

Details

Problem: Hardcoded Credentials - There are hardcoded plain-text user credentials in the auth service. While some credentials were removed, "alice" and "bob" credentials remain in the code.
Fix: Replace hardcoded credentials with environment variables or use a secure credential store.
Why: Having credentials in code is a security vulnerability that exposes user accounts if the code is compromised.

router = APIRouter()

 USER_DB = {
-    "alice": "password123",
-    "bob": "hunter2"
+    # User credentials should be loaded from a secure store or environment variables
 }

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We’d love your feedback! 🚀

Copy link

gitstream-cm bot commented May 10, 2025

This PR is missing a Jira ticket reference in the title or description.
Please add a Jira ticket reference to the title or description of this PR.

Copy link

gitstream-cm bot commented May 10, 2025

Hello vlussenburg 👋 Thanks for making your first PR, and welcome to our project!
Our mentor team has automatically been assigned to review this PR and guide you through the process.
Please reach out to that team if you have questions about the next steps.

Copy link

gitstream-cm bot commented May 10, 2025

Be a life saver 🛟 by adding a screenshot of the changes you made.

Copy link

gitstream-cm bot commented May 10, 2025

🥷 Code experts: cghyzel, amitmohleji

cghyzel, amitmohleji have most 👩‍💻 activity in the files.
cghyzel, amitmohleji have most 🧠 knowledge in the files.

See details

frontend/public/app.js

Activity based on git-commit:

cghyzel amitmohleji
MAY
APR
MAR 33 additions & 0 deletions
FEB
JAN
DEC

Knowledge based on git-blame:
amitmohleji: 100%

frontend/public/index.html

Activity based on git-commit:

cghyzel amitmohleji
MAY
APR
MAR 20 additions & 0 deletions
FEB
JAN
DEC

Knowledge based on git-blame:
amitmohleji: 100%

services/auth-python/app/auth.py

Activity based on git-commit:

cghyzel amitmohleji
MAY
APR
MAR
FEB 33 additions & 0 deletions
JAN
DEC

Knowledge based on git-blame:
cghyzel: 100%

services/billing-csharp/Controllers/BillingController.cs

Activity based on git-commit:

cghyzel amitmohleji
MAY
APR 45 additions & 0 deletions
MAR
FEB
JAN
DEC

Knowledge based on git-blame:
cghyzel: 100%

services/orders-java/src/main/java/com/example/orders/controller/OrderController.java

Activity based on git-commit:

cghyzel amitmohleji
MAY
APR 75 additions & 0 deletions
MAR
FEB
JAN
DEC

Knowledge based on git-blame:
cghyzel: 100%

To learn more about /:\ gitStream - Visit our Docs

@gitstream-cm gitstream-cm bot requested review from cghyzel and amitmohleji May 10, 2025 03:27
@vlussenburg vlussenburg deleted the code-update-standardization branch May 11, 2025 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants