Skip to content

[FEATURE] parse path parameters #34

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.

Sign up for GitHub

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

Jump to bottom
Merged
yabjames merged 22 commits into from
Dec 28, 2025
Merged

[FEATURE] parse path parameters #34

yabjames merged 22 commits into from
Dec 28, 2025

Conversation

@yabjames
Copy link
Owner

@yabjames yabjames commented Dec 28, 2025 •
edited
Loading

  • Reformat data structures into separate header files
  • Adds clang-format file for consistant formatting
  • Adds a feature to allow for path parameters using {}. (ex: /foo/{id})
  • Adds tests for the new path parameter parsing

@yabjames
Refactor parsing logic into multiple functions
0f7e668
@yabjames
Refactor: range for-loops, add const and static class methods where p…
788a8fb 
…ossible
@yabjames
Reformat using clang-format
c79ce8d
@yabjames
Reformat AtomicQueue.h using clang-format
4ef4a24
@yabjames
Separate parsing logic into a different class
98b8c50
@yabjames
Add a split_path method
cc63d52
@yabjames
Add testing for the HttpParser class
e767dbf
@yabjames
Reformat #include directives by removing relative file paths
5228670
@yabjames
Add condition for parsing '/' route
8d036f3
@yabjames
Implement a test for using split_path on "/" route
132bba0
@yabjames
Implement a way to match client request routes and open server routes
4db1a9c
@yabjames
Refactor structs into a utils header file
62131c2
@yabjames
Refactor data structures into new files and namespace
ee00d05
@yabjames
Implement a PathParams data structure
18c6ab6
@yabjames
Refactor match_route to only modify the request path params if there …
e6a906d 
…is a match
@yabjames
Refactor Route struct to a different file
faad4b1
@yabjames
Refactor handle_client to use the new route and path params matching …
4e70a7a 
…implementation
@yabjames
Add testing for path parameters
9a1f766
@yabjames
Update path parameter test
0abeb41
@yabjames
Add a test to handle no path parameters
cf9df2b
@yabjames
Add a function to the Response struct to convert it into a string
5ad24bd
@yabjames yabjames linked an issue Dec 28, 2025 that may be closed by this pull request
[FEATURE] Parse path parameters #32
Closed
@github-actions
Copy link

github-actions bot commented Dec 28, 2025

LCOV of commit 5ad24bd during server_workflows.yml #27

Summary coverage rate:
  lines......: 74.4% (1090 of 1465 lines)
  functions..: 82.6% (716 of 867 functions)
  branches...: no data found

Files changed coverage rate: n/a

@github-actions
Copy link

github-actions bot commented Dec 28, 2025

LCOV of commit 9608887 during server_workflows.yml #28

Summary coverage rate:
  lines......: 74.5% (1093 of 1468 lines)
  functions..: 82.6% (716 of 867 functions)
  branches...: no data found

Files changed coverage rate: n/a

@yabjames yabjames marked this pull request as ready for review December 28, 2025 19:35
@yabjames yabjames changed the title 32 feature parse path parameters [FEATURE] parse path parameters Dec 28, 2025
@yabjames yabjames merged commit b49f08a into main Dec 28, 2025
1 check passed
@yabjames yabjames deleted the 32-feature-parse-path-parameters branch December 28, 2025 19:36
@yabjames yabjames linked an issue Dec 28, 2025 that may be closed by this pull request
[FEATURE] Parse query parameters #33
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEATURE] Parse query parameters [FEATURE] Parse path parameters

2 participants

@yabjames