peru icon indicating copy to clipboard operation
peru copied to clipboard

Allow curl module to be used as http module

Open felipefoz opened this issue 3 years ago • 2 comments

As mentioned and briefly discussed in #195 by @oconnor663 and @colindean, the curl module today in peru is not actually using curl, which is confusing and can lead to wrong conclusions.

The proposal here is to create an alias for the curl module to be also recognized as http.

And then in future versions we keep only http as the module using the requests library.

felipefoz avatar Aug 31 '21 21:08 felipefoz

Could we be concrete about what confusions result from the current name. For example, what specific things does Curl do with e.g. your ~/.netrc file, where peru might surprise people by not doing that thing? Is there anything besides ~/.netrc?

oconnor663 avatar Sep 02 '21 15:09 oconnor663

Yes, I believe software should clearly inform its intent. In this case, the curl module make us think that curl is used for downloading the packages, as the git module uses git for downloading git repositories. Wouldn't it be a surprise for you that the git module actually was using other software than git?

That happened to me when I discovered that curl was not actually curl, while I haven't explored yet all that curl can do differently besides .netrc, as you mentioned.

The point is: the name is not suitable and even http might not be either, but we could be more clear on that, couldn't we?

felipefoz avatar Sep 02 '21 18:09 felipefoz