Skip to content

[πŸ› Bug]: Grid File download capability should be w3c compliant Β #11466

@titusfortner

Description

@titusfortner

What happened?

I'm sorry I didn't weigh in on #11277 before it was merged. I'm thinking of this specifically for how to get this properly consumed by the bindings as per #11443

2 things to bring the grid code more in line with other endpoints.

  1. Use POST if there is info to send to an endpoint instead of query string
  2. Return payload needs to be nested inside a value key.

Current implementation of file endpoint is:

POST /session/:sessionId:/se/file
file: <zippedFile>

To minimize confusion, I think the new feature should use files instead of file:

POST /session/:sessionId:/se/files
{fileName: <filename>}

and for #11458

GET /session/:sessionId:/se/files

How can we reproduce the issue?

n/a

Relevant log output

n/a

Operating System

n/a

Selenium version

trunk

What are the browser(s) and version(s) where you see this issue?

n/a

What are the browser driver(s) and version(s) where you see this issue?

n/a

Are you using Selenium Grid?

trunk

Metadata

Metadata

Assignees

Labels

B-gridEverything grid and server relatedI-defectSomething is not working as intended

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions