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

Stop scanning pages with a large number of inputs #257

Merged

Conversation

alistairjcbrown
Copy link
Member

Reviewer: @GioSensation @shakyShane
Asana: https://app.asana.com/0/1198964220583541/1203995347288602/f

Description

Stop scanning pages with a large number of inputs

Steps to test

  1. Add to extension
  2. Go to site with a large number of inputs (e.g. testing page - https://jsfiddle.net/alistairjcbrown/es506ubm/)
  3. Confirm that Dax does not show in the inputs2.
Large number of inputs Small number of inputs
Screenshot 2023-02-17 at 19 31 39 Screenshot 2023-02-17 at 19 31 54

Copy link
Collaborator

@shakyShane shakyShane left a comment

Choose a reason for hiding this comment

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

@alistairjcbrown thanks!

I've approved the code-changes, but I have to caveat the approval by saying I've not actually tested this in any of our browsers etc :)

@alistairjcbrown alistairjcbrown merged commit 645a5ec into main Feb 21, 2023
@alistairjcbrown alistairjcbrown deleted the abrown/stop-scanner-for-pages-with-lots-of-inputs branch February 21, 2023 11:50
CDRussell pushed a commit to duckduckgo/Android that referenced this pull request Mar 23, 2023
Task/Issue URL:
https://app.asana.com/0/1204250253793536/1204250253793536
Autofill Release:
https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/6.4.1


## Description
Updates Autofill to version
[6.4.1](https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/6.4.1).

### Autofill 6.4.0 and 6.4.1 release notes
## What's Changed

### Tooling and Tests
* Update integration tests for in-context signup by @alistairjcbrown in
duckduckgo/duckduckgo-autofill#254 and
duckduckgo/duckduckgo-autofill#255
* Update macOS browser repository ID in `asana-release.yml` by @ayoy in
duckduckgo/duckduckgo-autofill#259
* Use node v18 and update actions to read `.nvmrc` file by
@alistairjcbrown in
duckduckgo/duckduckgo-autofill#269
* Bugfix: Update release script to add checkout path to node version
file path by @alistairjcbrown in
duckduckgo/duckduckgo-autofill#286

### Source Code Updates
* Refactor in-context state to feature file by @shakyShane in
duckduckgo/duckduckgo-autofill#251
* Prevent duplicated schema types by using JSON schema throughout by
@shakyShane in
duckduckgo/duckduckgo-autofill#253
* Stop scanning pages with a large number of inputs by @alistairjcbrown
in duckduckgo/duckduckgo-autofill#257 and
duckduckgo/duckduckgo-autofill#262
* Ignore small email inputs by @alistairjcbrown in
duckduckgo/duckduckgo-autofill#261
* Fix username not saved by @GioSensation in
duckduckgo/duckduckgo-autofill#275
* Updates to in-context signup treatment by @alistairjcbrown and
@GioSensation in
duckduckgo/duckduckgo-autofill#284


**Full Changelog**:
duckduckgo/duckduckgo-autofill@6.3.0...6.4.1

## Steps to test
This release has been tested during autofill development. For smoke test
steps see [this
task](https://app.asana.com/0/1198964220583541/1200583647142330/f).

Co-authored-by: alistairjcbrown <alistairjcbrown@users.noreply.github.com>
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.

2 participants