Skip to content

Conversation

mengnankkkk
Copy link
Contributor

@mengnankkkk mengnankkkk commented Sep 18, 2025

Describe what this PR does / why we need it

from #2423

Does this pull request fix one issue?

Describe how you did it

Describe how to verify it

Special notes for reviews

@github-actions github-actions bot added area/graph SAA Grpah module area/studio SAA Studio module labels Sep 18, 2025
Copy link
Collaborator

@VLSMB VLSMB left a comment

Choose a reason for hiding this comment

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

There's no need to add information related to this node in Studio, and there's no corresponding node in Dify either. All you need to do is add an HTTP node that supports streaming operation. Please restore all code modifications for the studio section

@mengnankkkk
Copy link
Contributor Author

My CI test always shows that it is a CI environment configuration problem. How can I solve it?

@yuluo-yx
Copy link
Collaborator

  • 🛠️ Build and Test / test (pull_request)

pls check ci log
https://github.com/alibaba/spring-ai-alibaba/actions/runs/17824067433/job/50673270927?pr=2438

Error: -ai-alibaba-graph-core] [ERROR] Tests run: 18, Failures: 10, Errors: 0, Skipped: 0, Time elapsed: 0.387 s <<< FAILURE! -- in com.alibaba.cloud.ai.graph.streaming.StreamHttpNodeTest
Error: -ai-alibaba-graph-core] [ERROR] com.alibaba.cloud.ai.graph.streaming.StreamHttpNodeTest.testUrlValidation_ShouldRejectInternalAddress -- Time elapsed: 0.085 s
Error: -ai-alibaba-graph-core] [ERROR] com.alibaba.cloud.ai.graph.streaming.StreamHttpNodeTest.testUrlValidation_ShouldAllowInternalAddressWhenConfigured -- Time elapsed: 0.051 s
Error: -ai-alibaba-graph-core] [ERROR] com.alibaba.cloud.ai.graph.streaming.StreamHttpNodeTest.testAggregateMode -- Time elapsed: 0.030 s <<< FAILURE!
java.lang.AssertionError: 

Expecting actual:
  {"error"="StreamHttpNode[stream-http] failed: URL validation failed: Internal network access not allowed: localhost. Set allowInternalAddress=true to enable. (URL: http://localhost:47667/aggregate, Status: -1)", "streaming"=false, "timestamp"=1758187524364L}
to contain key:
  "data"
	at com.alibaba.cloud.ai.graph.streaming.StreamHttpNodeTest.lambda$testAggregateMode$9(StreamHttpNodeTest.java:232)
	at reactor.test.DefaultStepVerifierBuilder.lambda$consumeNextWith$1(DefaultStepVerifierBuilder.java:279)
	at reactor.test.DefaultStepVerifierBuilder$SignalEvent.test(DefaultStepVerifierBuilder.java:2289)
	at reactor.test.DefaultStepVerifierBuilder$DefaultVerifySubscriber.onSignal(DefaultStepVerifierBuilder.java:1529)
	at reactor.test.DefaultStepVerifierBuilder$DefaultVerifySubscriber.onExpectation(DefaultStepVerifierBuilder.java:1477)
	at reactor.test.DefaultStepVerifierBuilder$DefaultVerifySubscriber.onNext(DefaultStepVerifierBuilder.java:1146)
	at reactor.core.publisher.Operators$ScalarSubscription.request(Operators.java:2570)
	at reactor.test.DefaultStepVerifierBuilder$DefaultVerifySubscriber.onSubscribe(DefaultStepVerifierBuilder.java:1161)
	at reactor.core.publisher.FluxJust.subscribe(FluxJust.java:68)
	at reactor.core.publisher.Flux.subscribe(Flux.java:8891)
	at reactor.test.DefaultStepVerifierBuilder$DefaultStepVerifier.toVerifierAndSubscribe(DefaultStepVerifierBuilder.java:891)
	at reactor.test.DefaultStepVerifierBuilder$DefaultStepVerifier.verify(DefaultStepVerifierBuilder.java:831)
	at reactor.test.DefaultStepVerifierBuilder$DefaultStepVerifier.verify(DefaultStepVerifierBuilder.java:823)
	at reactor.test.DefaultStepVerifierBuilder.verifyComplete(DefaultStepVerifierBuilder.java:690)
	at com.alibaba.cloud.ai.graph.streaming.StreamHttpNodeTest.testAggregateMode(StreamHttpNodeTest.java:239)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

@github-actions github-actions bot removed the area/studio SAA Studio module label Sep 18, 2025
@mengnankkkk mengnankkkk requested a review from VLSMB September 18, 2025 13:09
@mengnankkkk mengnankkkk requested a review from VLSMB September 19, 2025 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/graph SAA Grpah module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants