Skip to content
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

Enhance bootup logging to include detailed rule information #1333

Merged
merged 35 commits into from
Jan 7, 2025

Conversation

t-burch
Copy link
Collaborator

@t-burch t-burch commented Nov 5, 2024

No description provided.

- Add additional logging when the router starts to display the number of rules and their details, including names and specific configurations for APIProxy and SOAPProxy.
- Implement helper methods to format rule display names and custom names for better clarity in logs.
- Update `InternalProxy` initialization to use `StringUtils` for key generation.
@t-burch t-burch requested a review from predic8 November 5, 2024 10:26
@t-burch t-burch linked an issue Nov 5, 2024 that may be closed by this pull request
@membrane-ci-server
Copy link

This pull request needs "/ok-to-test" from an authorized committer.

@t-burch
Copy link
Collaborator Author

t-burch commented Nov 8, 2024

/ok-to-test

christiangoerdes and others added 11 commits November 20, 2024 14:02
Implement methods to format and group OpenAPI records by directory. Enhance the display output to include detailed location information when OpenAPI records are present.
- Create a new test class `OpenAPI31ReferencesTest` for validating OpenAPI 3.1 references.
- Implement setup method to initialize the router and load OpenAPI specs.
- Update test cases to handle requests for both `/pets` and `/users` endpoints.
- Modify the existing OpenAPI specification to include server information in the YAML configuration.
# Conflicts:
#	core/src/test/java/com/predic8/membrane/core/openapi/serviceproxy/OpenAPI31ReferencesTest.java
#	core/src/test/resources/openapi/specs/oas31/request-reference.yaml
@predic8 predic8 marked this pull request as draft December 9, 2024 14:44
# Conflicts:
#	distribution/conf/proxies.xml
# Conflicts:
#	core/src/main/java/com/predic8/membrane/core/Router.java
#	core/src/main/java/com/predic8/membrane/core/proxies/RuleDisplayInfo.java
#	core/src/main/java/com/predic8/membrane/core/rules/InternalProxy.java
…e code clarity

- Remove redundant method for additional proxy display name and consolidate logic
- Enhance formatting for OpenAPI and WSDL information in log output
- Streamline location info formatting to improve readability
- Remove unused methods and imports to clean up the code
- Add the Cloneable interface to OpenAPISpec and Rewrite classes.
- Implement custom clone methods to ensure deep copying of mutable fields.
- Modify OpenAPIRecordFactory to utilize the cloned OpenAPISpec when creating records.
- Update logging in ProxyDisplayInfo to reflect enhanced proxy information.
@t-burch t-burch marked this pull request as ready for review January 7, 2025 09:31
…stead of 'api' method in internal service routing tests.
@predic8 predic8 merged commit 05400d8 into master Jan 7, 2025
3 of 4 checks passed
@predic8 predic8 deleted the #1297-bootup-api-info branch January 7, 2025 10:45
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.

Show info about apis at startup (Idea)
3 participants