Skip to content

Body Mixin text() is supposed to return USVString #39804

Description

@ronag

https://fetch.spec.whatwg.org/#body-mixin

@jasnell I think the Blob and consumer helper text() methods need to run through toUSVString which is used in the url module?

Activity

  1. changed the title [-]Body Mixin Text is supposed to return USVString[/-] [+]Body Mixin `text()` is supposed to return `USVString`[/+] on Aug 18, 2021
  2. jasnell commented on Aug 18, 2021

    @jasnell
    Member

    To be fully compliant, yes we likely need to.

  3. PurnashisHazra commented on Aug 23, 2021

    @PurnashisHazra

    Should I take this issue up?

  4. added
    good first issueIssues that are suitable for first-time contributors.
    streamIssues and PRs related to Node.js streams.
    web streamsIssues and PRs related to the Web Streams API.
    on Aug 24, 2021
  5. jimmywarting commented on Aug 27, 2021

    @jimmywarting

    just notice util.toUSVString came out in 16.8, good for ppl writing polyfills... got a side question: dose TextDecoder.decode() return a USVString? cuz that is what i have been using in some .text() methods.... This will also remove BOM

    I also wonder what the diffrent is between USVString and a "normal" string is and doing String(val) or ${val} to convert them
    Maybe stream-consumers text(iterable) should resolve with a USVString also?

  6. WenheLI commented on Sep 1, 2021

    @WenheLI

    Anyone currently working on this issue? If not, can I try to solve it?

  7. git-srinivas commented on Oct 3, 2021

    @git-srinivas
    Contributor

    Hi @ronag I am working on this issue. I could see the Blob contains the text() method. what is consumer helper here? can you please point the file or provide some more information about it?

  8. Mesteery commented on Oct 4, 2021

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueIssues that are suitable for first-time contributors.streamIssues and PRs related to Node.js streams.web streamsIssues and PRs related to the Web Streams API.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions