Skip to content

fix: non-processible error when empty body is sent to fastAPI (#10248)#10253

Merged
jordanrfrazier merged 1 commit into
release-1.6.5from
fix-fastapi-issue
Oct 13, 2025
Merged

fix: non-processible error when empty body is sent to fastAPI (#10248)#10253
jordanrfrazier merged 1 commit into
release-1.6.5from
fix-fastapi-issue

Conversation

@jordanrfrazier
Copy link
Copy Markdown
Collaborator

  • Fix non-processible erro when empty body is sent to fastAPI

  • Annotate FlowDataRequest with Body for embedding

  • Refine requestBody type for vertex order retrieval

  • [autofix.ci] apply automated fixes


Cherrypick from 5ea316d 5ea316d

* Fix non-processible erro when empty body is sent to fastAPI

* Annotate FlowDataRequest with Body for embedding

* Refine requestBody type for vertex order retrieval

* [autofix.ci] apply automated fixes

---------

Co-authored-by: Gabriel Luiz Freitas Almeida <gabriel@langflow.org>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
@github-actions github-actions Bot added the bug Something isn't working label Oct 13, 2025
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 13, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-fastapi-issue

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jordanrfrazier jordanrfrazier enabled auto-merge (squash) October 13, 2025 18:41
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Collaborator

@lucaseduoli lucaseduoli left a comment

Choose a reason for hiding this comment

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

LGTM! tested and working

@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label Oct 13, 2025
@jordanrfrazier jordanrfrazier added lgtm This PR has been approved by a maintainer and removed lgtm This PR has been approved by a maintainer labels Oct 13, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Frontend Unit Test Coverage Report

Coverage Summary

Lines Statements Branches Functions
Coverage: 9%
9.58% (2522/26307) 3.9% (793/20313) 4.04% (229/5660)

Unit Test Results

Tests Skipped Failures Errors Time
831 0 💤 0 ❌ 0 🔥 16.142s ⏱️

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 13, 2025

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (release-1.6.5@78b4134). Learn more about missing BASE report.

Files with missing lines Patch % Lines
.../queries/vertex/use-post-retrieve-vertex-order.tsx 0.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##             release-1.6.5   #10253   +/-   ##
================================================
  Coverage                 ?   35.78%           
================================================
  Files                    ?     1221           
  Lines                    ?    58297           
  Branches                 ?     5549           
================================================
  Hits                     ?    20863           
  Misses                   ?    37273           
  Partials                 ?      161           
Flag Coverage Δ
backend 55.73% <ø> (?)
frontend 8.81% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
.../queries/vertex/use-post-retrieve-vertex-order.tsx 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jordanrfrazier jordanrfrazier merged commit 06c9ab8 into release-1.6.5 Oct 13, 2025
72 of 73 checks passed
@jordanrfrazier jordanrfrazier deleted the fix-fastapi-issue branch October 13, 2025 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants