Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Added local to SanitizationWhitelist #8787

Closed
wants to merge 180 commits into from
Closed

Added local to SanitizationWhitelist #8787

wants to merge 180 commits into from

Conversation

jorgecasar
Copy link

HTML5 apps on BlackBerry 10 devices use "local" as protocol for local files.

HTML5 apps on BlackBerry 10 devices use "local" as protocol for local files.
@mary-poppins
Copy link

I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS.

Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match.

If you signed the CLA as a corporation, please let us know the company's name.

Thanks a bunch!

PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR.
PS2: If you are a Googler, please sign the CLA as well to simplify the CLA verification process.

@jorgecasar
Copy link
Author

I've just signed the CLA.

@mary-poppins
Copy link

CLA signature verified! Thank you!

Someone from the team will now triage your PR and it will be processed based on the determined priority (doc updates and fixes with tests are prioritized over other changes).

Added test for ´local´'s protocol to be sure `local:///` urls are not sanitized neither `src` nor `href`.
@jeffbcross
Copy link
Contributor

Hi @jorgecasar, a few housekeeping items:

  • Please squash your commits into a single commit
  • Travis failed because of some jshint errors. Please fix those and amend your commit
  • Please update your commit message to comply with our contribution guidelines
  • Please provide a link in the commit showing documentation regarding this protocol

@jeffbcross jeffbcross added this to the Backlog milestone Sep 3, 2014
@jeffbcross jeffbcross removed their assignment Sep 3, 2014
jeffbcross and others added 10 commits September 11, 2014 11:23
In some cases, the type of Error thrown by minErr is meaningful, such as in $q where a TypeError
is sometimes required. This fix allows providing an error constructor as the second argument to
minErr, which will be used to construct the error that gets returned by the factory function.
jQuery was not included in e2e tests, but we did not notice it
as Angular fell back to jqlite…
Angular used to always use the browser timezone for
`dateFilter`. An additional parameter was added to allow to use
`UTC` timezone instead.

Related to #8447.
BREAKING CHANGE:

According to the HTML5 spec `input[time]` should create dates
based on the year 1970 (used to be based on the year 1900).

Related to #8447.
Angular used to always use the browser timezone when parsing
`input[date]`, `input[time]`, … The timezone can now be changed
to `UTC` via `ngModelOptions`.

Closes #8447.
…-local]`

The HTML5 spec allows to use seconds for `input[time]` and `input[datetime-local]`,
even though they are not displayed by all browsers.

Related to #8447.
jeffbcross and others added 28 commits September 11, 2014 11:23
…emplate"

This reverts commit 6d1e7cd.

This commit was causing breakages because of its assumption that transcluded
content would be handled predictably, i.e. with ngTransclude, whereas many
use cases involve manipulating transcluded content in linking functions.
- define `ngModelGet` and `ngModelSet` to already use
  the getter/setter semantics, so the rest of the code does
  not need to care about it.
- remove `ctrl.$$invalidModelValue` to simplify the internal logic
If the view value changed in the first digest and there are async validators,
the view value was never applied to the model after the validators were
resolved. Only important for tests.
This option allows to write invalid values to the model instead of having them become undefined.
Use this together with calling `ctrl.$setValidity` directly for displaying errors
from serverside validation.

Closes #8290
Closes #8313
Calling `ctrl.$setValidity()` with a an error key that
does not belong to a validator in `ctrl.$validator` should
not result in setting the model to `undefined` on the next
input change. This bug was introduced in 1.3.0-beta.12.

Closes #8357
Fixes #8080
This changes the example JSON to be the actual bower.json for angular-phonecat,
with name set to angular-phonecat.
Previously, if you bound a `Date` object to `<input type="time">`,
whenever you changed the time, the day, month, and year fields of
the new resulting bound `Date` object would be reset. Now fields
not modified by bound time input elements are copied to the new
resulting object.

Same for input types of `month`, `week`, etc.

Closes #6666
Fix typo at i18n generation code. This would remove the
property `macFrac` that has no meaning from all the
generated locales
The error message should help identify the problem. To do so, more details need to be provided.

Closes #9005
Closes #9010
The conclusion table incorrectly states that services can not create functions.
New table row added to separate "can create functions" and "can create primitives".
…Script

Adds missing number format and symbol rules to Closure slurper script.

Closes #6179
Closes #9013
Fixes number formatting and symbols for many locales.
Adds support for additional locales.

Closes #8931
Closes #8583
Closes #7799
@jorgecasar
Copy link
Author

Sorry, Something goes wrong rebasing my fork

@jorgecasar jorgecasar closed this Sep 11, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.