Skip to content

[webview_flutter] Support for handling basic authentication requests (Platform Interface) - #5362

Merged
auto-submit[bot] merged 48 commits into
flutter:mainfrom
andreisas06:webview-auth-request-platform-interface
Nov 20, 2023
Merged

[webview_flutter] Support for handling basic authentication requests (Platform Interface)#5362
auto-submit[bot] merged 48 commits into
flutter:mainfrom
andreisas06:webview-auth-request-platform-interface

Conversation

@JeroenWeener

Copy link
Copy Markdown
Contributor

Adds the platform interface implementation for basic http authentication.

This PR is part of a series of PRs that aim to closeflutter/flutter#83556.
The PR that contains all changes can be found at #4140.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use dart format.)
  • I signed the CLA.
  • The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
  • I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

JeroenWeenerand others added 5 commits November 17, 2023 13:58
…b/src/types/webview_credential.dart
Co-authored-by: Maurice Parrish <10687576+bparrishMines@users.noreply.github.com>
…b/src/types/http_auth_request.dart
Co-authored-by: Maurice Parrish <10687576+bparrishMines@users.noreply.github.com>
…b/src/types/http_auth_request.dart
Co-authored-by: Maurice Parrish <10687576+bparrishMines@users.noreply.github.com>
@JeroenWeener

Copy link
Copy Markdown
ContributorAuthor

Thanks @bparrishMines, I updated the branch based on your comments.

@stuartmorgan-gstuartmorgan-g left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM with one question.

sdk: flutter
meta: ^1.7.0
plugin_platform_interface: ^2.1.0
plugin_platform_interface: ^2.1.6

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why is this changing? I don't think anything changed in that version that we could be depending on.

@stuartmorgan-gstuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 20, 2023
@stuartmorgan-g

Copy link
Copy Markdown
Collaborator

Overriding the out-of-date tree check; the dashboard is for some reason showing that run 490 is the last run of the failing test, but GitHub UI is correctly registering run 491 which is a re-run that passed.

@stuartmorgan-gstuartmorgan-g added the emergency Override tree-status signal (land even with closed tree), combine with the autosubmit label. label Nov 20, 2023
@auto-submit
auto-submitBot merged commit c5443ad into flutter:mainNov 20, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 20, 2023
fluttermirroringbot pushed a commit to flutter/flutter that referenced this pull request Nov 20, 2023
flutter/packages@07b4b29...c5443ad
2023-11-20 JeroenWeener@users.noreply.github.com [webview_flutter] Support for handling basic authentication requests (Platform Interface) (flutter/packages#5362)
2023-11-18 engine-flutter-autoroll@skia.org Roll Flutter from 53a57ad to 6cf9ab0 (31 revisions) (flutter/packages#5426)
2023-11-18 kevmoo@users.noreply.github.com [shared_preferences_web] migrate to pkg:web (flutter/packages#5414)
2023-11-18 stuartmorgan@google.com [ci] Roll minimum allowable Flutter to 3.10 (flutter/packages#5425)
2023-11-18 43054281+camsim99@users.noreply.github.com [path_provider_android] Run tests on AVDs running Android 34 (flutter/packages#5222)
2023-11-17 tarrinneal@gmail.com [pigeon] isEnum, isClass, fix swift casting, default values, optional method arguments, named method arguments (flutter/packages#5355)
2023-11-17 stuartmorgan@google.com [plugin_platform_interface] Switch mixin to `mixin class` (flutter/packages#5420)
2023-11-17 stuartmorgan@google.com [go_router] Fixes use of `Iterable` (flutter/packages#5421)
2023-11-17 stuartmorgan@google.com [pigeon] Adds `analyzer` 6.x compatibility (flutter/packages#5418)
2023-11-17 stuartmorgan@google.com Update release step to 3.16 (flutter/packages#5416)
2023-11-17 kevmoo@users.noreply.github.com [file_selector_web] migrate to pkg:web (flutter/packages#5413)
2023-11-16 stuartmorgan@google.com [video_player] Improve macOS frame management (flutter/packages#5078)
2023-11-16 engine-flutter-autoroll@skia.org Roll Flutter from e8c2bb1 to 53a57ad (39 revisions) (flutter/packages#5412)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@reidbakerreidbaker mentioned this pull request Jan 12, 2024
auto-submitBot pushed a commit that referenced this pull request Feb 1, 2024
…5727)
## Description
This pull request exposes the Android and iOS HTTP Basic Authentication feature to users of the `webview_flutter` plugin. It is the final PR in a sequence of PRs. Previous PRs are #5362, #5454 and #5455.
Issues fixed by PR:
Closesflutter/flutter#83556
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…(Platform Interface) (flutter#5362)
Adds the platform interface implementation for basic http authentication.
This PR is part of a series of PRs that aim to closeflutter/flutter#83556.
The PR that contains all changes can be found at flutter#4140.
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…lutter#5727)
## Description
This pull request exposes the Android and iOS HTTP Basic Authentication feature to users of the `webview_flutter` plugin. It is the final PR in a sequence of PRs. Previous PRs are flutter#5362, flutter#5454 and flutter#5455.
Issues fixed by PR:
Closesflutter/flutter#83556
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
…(Platform Interface) (flutter#5362)
Adds the platform interface implementation for basic http authentication.
This PR is part of a series of PRs that aim to closeflutter/flutter#83556.
The PR that contains all changes can be found at flutter#4140.
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
…lutter#5727)
## Description
This pull request exposes the Android and iOS HTTP Basic Authentication feature to users of the `webview_flutter` plugin. It is the final PR in a sequence of PRs. Previous PRs are flutter#5362, flutter#5454 and flutter#5455.
Issues fixed by PR:
Closesflutter/flutter#83556
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit AppemergencyOverride tree-status signal (land even with closed tree), combine with the autosubmit label.p: webview_flutter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[webview_flutter] There is no callback for "Basic Authentication" on navigating to a URI

3 participants

@JeroenWeener@stuartmorgan-g@bparrishMines