Skip to content

Conversation

@yqw570994511
Copy link
Contributor

Support for tracking in spring gateway versions 4.1.2 and above

see:apache/skywalking#12925

@wu-sheng wu-sheng added this to the 9.4.0 milestone Jan 10, 2025

@Override
public boolean isOverrideArgs() {
return true;
Copy link
Member

Choose a reason for hiding this comment

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

Why true? I can't see you override the args in the interceptor.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Why true? I can't see you override the args in the interceptor.

You are right, it should not be true

@wu-sheng
Copy link
Member

Error: error: unable to create file apm-sniffer/optional-plugins/optional-spring-plugins/optional-spring-cloud/gateway-4.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/spring/cloud/gateway/v412x/HttpClientFinalizerResponseConnectionV412Interceptor.java: Filename too long

It seems we face Windows limitations. Usually, we need to adjust the configuration of Windows.

@yqw570994511
Copy link
Contributor Author

Error: error: unable to create file apm-sniffer/optional-plugins/optional-spring-plugins/optional-spring-cloud/gateway-4.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/spring/cloud/gateway/v412x/HttpClientFinalizerResponseConnectionV412Interceptor.java: Filename too long

It seems we face Windows limitations. Usually, we need to adjust the configuration of Windows.

Will it affect the submission of this file?

@wu-sheng
Copy link
Member

The new added file seems to exceed length limitation.

@yqw570994511
Copy link
Contributor Author

The new added file seems to exceed length limitation.

How should we handle this issue? Should we modify it to a shorter file name? But I don't have a Windows environment to verify how suitable it is. Do you have any good suggestions

@wu-sheng
Copy link
Member

Error: error: unable to create file apm-sniffer/optional-plugins/optional-spring-plugins/optional-spring-cloud/gateway-4.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/spring/cloud/gateway/v412x/HttpClientFinalizerResponseConnectionV412Interceptor.java: Filename too long

It seems we face Windows limitations. Usually, we need to adjust the configuration of Windows.

@kezhenxu94 Do you have any recommendation to fix this in CI?

@kezhenxu94
Copy link
Member

Error: error: unable to create file apm-sniffer/optional-plugins/optional-spring-plugins/optional-spring-cloud/gateway-4.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/spring/cloud/gateway/v412x/HttpClientFinalizerResponseConnectionV412Interceptor.java: Filename too long

It seems we face Windows limitations. Usually, we need to adjust the configuration of Windows.

@kezhenxu94 Do you have any recommendation to fix this in CI?

@yqw570994511 Can you please separate the command I added for windows only? Only add that command for windows platform. I'm not with my laptop now

@yqw570994511
Copy link
Contributor Author

Error: error: unable to create file apm-sniffer/optional-plugins/optional-spring-plugins/optional-spring-cloud/gateway-4.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/spring/cloud/gateway/v412x/HttpClientFinalizerResponseConnectionV412Interceptor.java: Filename too long

It seems we face Windows limitations. Usually, we need to adjust the configuration of Windows.

@kezhenxu94 Do you have any recommendation to fix this in CI?

@yqw570994511 Can you please separate the command I added for windows only? Only add that command for windows platform. I'm not with my laptop now

I'd love to help, but I'm not familiar with this and don't know how to operate it

@kezhenxu94
Copy link
Member

Error: error: unable to create file apm-sniffer/optional-plugins/optional-spring-plugins/optional-spring-cloud/gateway-4.x-plugin/src/main/java/org/apache/skywalking/apm/plugin/spring/cloud/gateway/v412x/HttpClientFinalizerResponseConnectionV412Interceptor.java: Filename too long

It seems we face Windows limitations. Usually, we need to adjust the configuration of Windows.

@kezhenxu94 Do you have any recommendation to fix this in CI?

@yqw570994511 Can you please separate the command I added for windows only? Only add that command for windows platform. I'm not with my laptop now

I'd love to help, but I'm not familiar with this and don't know how to operate it

Never mind. I've fixed it

@wu-sheng
Copy link
Member

@kezhenxu94 notice this

Invalid workflow file: .github/workflows/ci.yaml#L56
The workflow is not valid. .github/workflows/ci.yaml (Line: 56, Col: 13): Unexpected symbol: '"windows"'. Located at position 14 within expression: matrix.os == "windows"

@wu-sheng wu-sheng merged commit 3549537 into apache:main Jan 11, 2025
194 checks passed
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.

3 participants