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

Fix the connectionInfo getter #1163

Merged

brianquinlan merged 1 commit into dart-lang:master from brianquinlan:http_profile_connection_info

Mar 27, 2024

Collaborator

brianquinlan commented Mar 27, 2024

HttpClientRequestProfile.connectionInfo was using the wrong map location to fetch it's data

I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.


          Fix the connectionInfo getter

10e6b4f

brianquinlan requested a review from derekxu16

March 27, 2024 20:12

derekxu16 approved these changes

View reviewed changes

brianquinlan merged commit 9fddb1c into dart-lang:master

28 checks passed

copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request


          [deps] rev dartdoc, http

3b23ffe

Revisions updated by `dart tools/rev_sdk_deps.dart`.

dartdoc (https://github.com/dart-lang/dartdoc/compare/79c1675..bf6080c):
  bf6080c8  2024-03-27  Sam Rawlins  Remove unnecessary EnclosedElement mixin (dart-lang/dartdoc#3736)
  c5bb9066  2024-03-26  Sam Rawlins  Bump to 8.0.8; fix changelog wrapping (dart-lang/dartdoc#3735)
  4e096f84  2024-03-26  Sam Rawlins  Format parameters better (dart-lang/dartdoc#3731)

http (https://github.com/dart-lang/http/compare/7949d6f..280d361):
  280d361  2024-03-27  Derek Xu  [package:http_profile] Expand README.md (dart-lang/http#1162)
  9fddb1c  2024-03-27  Brian Quinlan  Fix the connectionInfo getter (dart-lang/http#1163)
  9f47439  2024-03-27  Derek Xu  [package:http_profile] Make connectionInfo a top-level field of HttpClientRequestProfile (dart-lang/http#1160)

Change-Id: Ib50584ef7d7914ab262bb01eeb087e44c34e4dc7
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/360240
Auto-Submit: Devon Carew <[email protected]>
Commit-Queue: Konstantin Shcheglov <[email protected]>
Reviewed-by: Konstantin Shcheglov <[email protected]>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet