Skip to content

Integrate curl-impersonate on the Docker file#2925

Closed
somini wants to merge 1 commit intoRSS-Bridge:masterfrom
somini:patch-1
Closed

Integrate curl-impersonate on the Docker file#2925
somini wants to merge 1 commit intoRSS-Bridge:masterfrom
somini:patch-1

Conversation

@somini
Copy link
Contributor

@somini somini commented Jul 16, 2022

I don't have much Docker experience, this is just a hack.

I also don't know how to use a variable to store the version in a single place, so that we can update it more easily.

See #2547

I don't have much Docker experience, this is just a hack.

I also don't know how to use a variable to store the version in a single place, so that we can update it more easily.

See RSS-Bridge#2547
@somini
Copy link
Contributor Author

somini commented Jul 16, 2022

The Dockerfile is not being rebuilt, from what I can tell. So this should not be merged without testing.

@somini somini changed the title Integrate curl-impersonate on the Docker file Draft: Integrate curl-impersonate on the Docker file Jul 16, 2022
@somini somini marked this pull request as draft July 16, 2022 10:11
@somini somini changed the title Draft: Integrate curl-impersonate on the Docker file Integrate curl-impersonate on the Docker file Jul 16, 2022
@Bockiii
Copy link
Contributor

Bockiii commented Jul 16, 2022

One of the immediate issues: There is no armv7 build, so this option cant be used in that image.

I'll check for normal amd usage now.

@wrobelda
Copy link
Contributor

wrobelda commented Jul 28, 2022

I proposed (#2941) a similar solution, except by leveraging multi-stage building, which solves the complexity of the system architectures.

@dvikan
Copy link
Contributor

dvikan commented Dec 13, 2022

See #2941

@dvikan dvikan closed this Dec 13, 2022
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.

4 participants