Skip to content

Add initial support for app: URL#5

Closed
marcoscaceres wants to merge 1 commit intowhatwg:masterfrom
marcoscaceres:app-url
Closed

Add initial support for app: URL#5
marcoscaceres wants to merge 1 commit intowhatwg:masterfrom
marcoscaceres:app-url

Conversation

@marcoscaceres
Copy link
Member

The app: URL is used by FxOS and Intel's cross-walk. The SysApps WG would like to standardize this. Instead of monkey patching, I thought I would try to add it here.

@marcoscaceres
Copy link
Member Author

ping

@annevk
Copy link
Member

annevk commented Apr 25, 2014

Is this really how we want this to work?

Also, how is this observable from the web?

@marcoscaceres
Copy link
Member Author

Is this really how we want this to work?

I think so, yes. But suggestions of improvements are always welcome.

Also, how is this observable from the web?

Origin header.

@annevk
Copy link
Member

annevk commented May 1, 2014

That's not really what observable from the web was intended to mean. If I create a custom script that spits out "Origin: I'm being silly" that's no grounds for standardization either.

If this is only for walled garden content "monkey patching" might be right approach.

@marcoscaceres
Copy link
Member Author

Ok, no probs. I can monkey patch this (was actually what I expected). I just wanted to go through the right channel and follow the process from your post.

yutakahirano added a commit that referenced this pull request Jun 23, 2020
# This is the 1st commit message:

# This is a combination of 23 commits.
# This is the 1st commit message:

Integrate CORP and COEP

This is part of the introduction of COEP
(whatwg/html#5454). The CORP check now takes
COEP into account. Also, responses coming from service workers
are checked.

# This is the commit message #2:

Update fetch.bs

Co-authored-by: Domenic Denicola <d@domenic.me>
# This is the commit message #3:

Update fetch.bs

Co-authored-by: Domenic Denicola <d@domenic.me>
# This is the commit message #4:

fix

# This is the commit message #5:

fix

# This is the commit message #6:

fix

# This is the commit message #7:

fix

# This is the commit message #8:

fix

# This is the commit message #9:

fix

# This is the commit message #10:

fix

# This is the commit message #11:

fix

# This is the commit message #12:

fix

# This is the commit message #13:

fix

# This is the commit message #14:

fix

# This is the commit message #15:

fix

# This is the commit message #16:

fix

# This is the commit message #17:

fix

# This is the commit message #18:

Update fetch.bs

Co-authored-by: Anne van Kesteren <annevk@annevk.nl>
# This is the commit message #19:

Update fetch.bs

Co-authored-by: Anne van Kesteren <annevk@annevk.nl>
# This is the commit message #20:

fix

# This is the commit message #21:

fix

# This is the commit message #22:

fix

# This is the commit message #23:

fix

# This is the commit message #2:

fix
@noamr noamr mentioned this pull request Jun 14, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants