Replace hackney with httpc #311

whatyouhide · 2023-06-20T10:08:41Z

This is the start of the work proposed in #231.

The main missing thing is Security™, that is, I need to add all the right SSL options to make the :httpc request secure. I'll work on that this afternoon if there is interest in this, @parroty?

Btw @parroty, also feel free to send me an email (email is on my GitHub profile) or write me a DM on Twitter to chat about helping out with maintaining this library 🙃

parroty · 2023-06-20T12:23:33Z

Thanks. It's a little hectic week for me, but try to send message on the weekend / next week.

whatyouhide · 2023-06-30T09:24:34Z

@parroty hello! Any news on this? 🙃

whatyouhide · 2023-07-09T07:57:25Z

@parroty ping 🙃 Would love to get an email and chat! 😉

whatyouhide · 2023-07-25T08:30:16Z

@parroty ping 🏓 🙃

whatyouhide · 2023-07-26T16:52:52Z

@parroty I added SSL support here too, based on esbuild.

parroty · 2023-07-27T18:58:08Z

Sorry being slow to respond. I am seeing actions errors, but haven't been able look into it. Do you have any idea?

whatyouhide · 2023-07-28T06:01:46Z

@parroty let's start out with #316 😄

whatyouhide · 2023-08-01T10:15:48Z

@parroty finally fixed tests in CI on the supported versions, so this is ready to merge!!! Exciting times ahead. 💟

whatyouhide · 2023-08-02T07:35:28Z

.github/workflows/tests.yml

@@ -25,8 +25,8 @@ jobs:
      - uses: actions/cache@v3
        with:
         path: deps
-         key: ${{ runner.os }}-mix-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
+         key: ${{ runner.os }}-${{ matrix.elixir }}-otp${{ matrix.otp }}-mix-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}


@parroty having a single cache for all OTP versions causes issues when testing across different versions. Most projects I've used actually go with a per-version cache, so this is what this change is.

whatyouhide · 2023-08-07T09:05:29Z

@parroty any news on this? 🙃

whatyouhide · 2023-08-08T17:09:20Z

Tested this out in Xandra, works good 🙃

whatyouhide · 2023-08-09T05:53:55Z

Woohooo! Thanks @parroty. Looking forward to a release 😉

gmile · 2023-08-09T08:33:20Z

lib/excoveralls/poster.ex

-    Application.ensure_all_started(:hackney)
+  defp upload_zipped_json(content, options) do
+    Application.ensure_all_started(:ssl)
+    Application.ensure_all_started(:httpc)


Out of curiosity, why does :httpc needs to be started explicitly? Starting :inets.start() and :ssl.start() typically was sufficient for me in the past 🤔

It probably is sufficient, I just overlooked this. PRs welcome 😉

Thanks for the comment. Do you see specific negative implication for this extra start logic? (Just a small overhead?).

I'll be publishing the release later today (after going back home), but wondering whether to update this before the release.

@parroty httpc is not an app. Just merge #318 before releasing 😉

Tests are passing, but CI is failing due to the old excoveralls issue seen before in parroty/excoveralls#254 However, the latest workarounds noted on that issue are for earlier versions of excoveralls that still used hackney instead of httpc (cf. parroty/excoveralls#311). So it's not clear that there are any good/known workarounds, as in the old issue. In particular, we should be running the latest castore version, and I don't think installing/updating certifi is going to do anything. The tests seem to pass on the other Elixir versions in the matrix, but they're all running at least OTP 23. So I'm wondering if OTP 23 would unstick Elixir 1.12 & 1.13. Not ideal, but worth a shot.

* Cobertura now handles defprotocol and defimpl definitions (parroty#306) * Cobertura now handles defprotocol and defimpl definitions * Cobertura, catch all if module type is unknown * Bump version and update CHANGELOG * Add Cobertura docs to README.md (parroty#312) * Update Elixir requirement to 1.11+ (parroty#316) * Update Elixir requirement to 1.11+ * Update deps * Replace hackney with httpc (parroty#311) * Replace hackney with httpc * SSL options * FIXUP * Cache fixed * Aaaah, caching again * FIXUP * Add missing apps to :extra_applications * Add better check for :public_key * Bump version and update CHANGELOG * Fix lcov 2.0 source file handling (parroty#315) Prior to 2.0 being released, `genhtml` was much better about handling source files from the following paths: * `test/support/some_helper.ex` * `lib/foo/bar.ex` But after 2.0 was released, when rendering with `genhtml` the paths would be mangled and look like the following: * `test/support/test/support/some_helper.ex` * `lib/foo/lib/foo/bar.ex` I have tried in vain with many permutations of `--prefix` when running `genhtml` but the ultimate fix that made all of this go away was using the absolute path for the source file (`SF`). * Update CHANGELOG * Remove erroneous line in ExCoveralls.poster (parroty#318) * Import `.coverdata` after test run and improve documentation (parroty#309) * Import `.coverdata` after test run and improve docs * Update README table of contents * Update README based on review suggestion Co-authored-by: Alberto Sartori <alberto.sartori.as@gmail.com> --------- Co-authored-by: Alberto Sartori <alberto.sartori.as@gmail.com> * Update CHANGELOG * Accept custom http options (parroty#319) * Accept custom http options * Add HTTP options docs to README * Bump version and update CHANGELOG * Always floor coverage instead of rounding (parroty#310) * Always floor coverage instead of rounding We do not want to report a 100% coverage when there are lines that are not covered. * Add option to restore previous ceil coverage behaviour * Bump version and update CHANGELOG * Update README examples (parroty#320) * Use explicit steps to remove 1.16 deprecation warning (parroty#322) * Update CHANGELOG and bump version * Detect and warn about incorrectly used ignore-comments (parroty#325) * Detect and warn about incorrectly used ignore-comments Resolves parroty#197. * Keep existing ignoring behavior * Improve formatting * Test more ignore-related warnings * Remove warning in the case of ignore-next-line at the EOF * Test the warning output * Add a changelog entry * Adjust test descriptions --------- Co-authored-by: Roman <205906+RKushnir@users.noreply.github.com> * Fix Elixir 1.17 single-quoted string warning (parroty#327) * Bump version and update CHANGELOG * add missing step for Cobertura's range (parroty#329) Without this change, the output is full of warnings such as ``` warning: negative steps are not supported in String.slice/2, pass 44..-1//1 instead ``` * Update CHANGELOG and version * Revert "add missing step for Cobertura's range (parroty#329)" (parroty#330) This reverts commit 00a96c4. * Revert version change * Update Range to use function syntax (parroty#332) * add missing step for Cobertura's range * Update Range to use function syntax * run tests in elixir 1.17 * fix test with relative path --------- Co-authored-by: parroty <parroty@users.noreply.github.com> * Update CHANGELOG and version --------- Co-authored-by: Rodrigue Villetard <rodrigue@villetard.tech> Co-authored-by: parroty <parroty@users.noreply.github.com> Co-authored-by: Artem Solomatin <artem00298@gmail.com> Co-authored-by: Andrea Leopardi <an.leopardi@gmail.com> Co-authored-by: Matthew Johnston <warmwaffles@gmail.com> Co-authored-by: Zach Allaun <zach.allaun@gmail.com> Co-authored-by: Alberto Sartori <alberto.sartori.as@gmail.com> Co-authored-by: Victor Rodrigues <rodrigues@users.noreply.github.com> Co-authored-by: gitneko <67227083+gitneko@users.noreply.github.com> Co-authored-by: Roman <RKushnir@users.noreply.github.com> Co-authored-by: Roman <205906+RKushnir@users.noreply.github.com> Co-authored-by: Hans Krutzer <git@pixelspaceships.com> Co-authored-by: Juan Peri <eternoperegrino@gmail.com> Co-authored-by: Kenta Nakase <1172471+parroty@users.noreply.github.com>

parroty mentioned this pull request Aug 1, 2023

Update Elixir requirement to 1.11+ #316

Merged

whatyouhide added 2 commits August 1, 2023 11:10

Replace hackney with httpc

0e8574c

SSL options

d7f525e

whatyouhide force-pushed the httpc branch from bdd5cb8 to d7f525e Compare August 1, 2023 09:10

whatyouhide marked this pull request as ready for review August 1, 2023 09:10

whatyouhide added 4 commits August 1, 2023 11:24

FIXUP

abd7dbf

Cache fixed

c42abb6

Aaaah, caching again

5ad9e5f

FIXUP

6275ab6

whatyouhide commented Aug 2, 2023

View reviewed changes

whatyouhide added 2 commits August 7, 2023 18:24

Add missing apps to :extra_applications

baff737

Add better check for :public_key

f2161dc

parroty merged commit e41649c into parroty:master Aug 9, 2023

whatyouhide deleted the httpc branch August 9, 2023 05:53

gmile reviewed Aug 9, 2023

View reviewed changes

whatyouhide mentioned this pull request Aug 9, 2023

Remove erroneous line in ExCoveralls.poster #318

Merged

albertored mentioned this pull request Aug 9, 2023

Make :hackney an optional dependency #231

Closed

ajvondrak mentioned this pull request Jun 10, 2024

(ExCoveralls.ReportUploadError) Failed to upload the report to 'https://coveralls.io', reason: tls_alert #254

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Replace hackney with httpc #311

Replace hackney with httpc #311

whatyouhide commented Jun 20, 2023

parroty commented Jun 20, 2023 •

edited

Loading

whatyouhide commented Jun 30, 2023

whatyouhide commented Jul 9, 2023

whatyouhide commented Jul 25, 2023

whatyouhide commented Jul 26, 2023

parroty commented Jul 27, 2023

whatyouhide commented Jul 28, 2023

whatyouhide commented Aug 1, 2023

whatyouhide Aug 2, 2023

whatyouhide commented Aug 7, 2023

whatyouhide commented Aug 8, 2023

whatyouhide commented Aug 9, 2023

gmile Aug 9, 2023

whatyouhide Aug 9, 2023

parroty Aug 9, 2023 •

edited

Loading

whatyouhide Aug 9, 2023

Replace hackney with httpc #311

Replace hackney with httpc #311

Conversation

whatyouhide commented Jun 20, 2023

parroty commented Jun 20, 2023 • edited Loading

whatyouhide commented Jun 30, 2023

whatyouhide commented Jul 9, 2023

whatyouhide commented Jul 25, 2023

whatyouhide commented Jul 26, 2023

parroty commented Jul 27, 2023

whatyouhide commented Jul 28, 2023

whatyouhide commented Aug 1, 2023

whatyouhide Aug 2, 2023

Choose a reason for hiding this comment

whatyouhide commented Aug 7, 2023

whatyouhide commented Aug 8, 2023

whatyouhide commented Aug 9, 2023

gmile Aug 9, 2023

Choose a reason for hiding this comment

whatyouhide Aug 9, 2023

Choose a reason for hiding this comment

parroty Aug 9, 2023 • edited Loading

Choose a reason for hiding this comment

whatyouhide Aug 9, 2023

Choose a reason for hiding this comment

parroty commented Jun 20, 2023 •

edited

Loading

parroty Aug 9, 2023 •

edited

Loading