Skip to content

fix mistakes in docstrings #13603

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

Merged
mattklein123 merged 1 commit into from
Oct 16, 2020
+3 −3
Merged

fix mistakes in docstrings #13603

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions source/common/http/path_utility.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,15 +12,15 @@ namespace Http {
*/
class PathUtil {
public:
// Returns if the normalization succeeds.
// If it is successful, the path header in header path will be updated with the normalized path.
// Returns true if the normalization succeeds.
// If it is successful, the path header will be updated with the normalized path.
// Requires the Path header be present.
static bool canonicalPath(RequestHeaderMap& headers);
// Merges two or more adjacent slashes in path part of URI into one.
// Requires the Path header be present.
static void mergeSlashes(RequestHeaderMap& headers);
// Removes the query and/or fragment string (if present) from the input path.
// For example, this function returns "/data" for the input path "/data#fragment?param=value".
// For example, this function returns "/data" for the input path "/data?param=value#fragment".
static absl::string_view removeQueryAndFragment(const absl::string_view path);
};

Expand Down