GithubHelp home page GithubHelp logo

Comments (32)

slince avatar slince commented on July 3, 2024 9

@krwu

其实既然决定使用composer分发代码,那一切按照composer的要求去做是最合适的;
还是建议你们的同事:

  • 移除src 下 guzzle以及symfony目录
  • 更新guzzle为guzzlehttp
  • .idea 目录移除
  • 补充单元测试

腾讯这么大一个厂不能太随心所欲,不讲究开源质量实在是说不过去。

from cos-php-sdk-v5.

zuoRambo avatar zuoRambo commented on July 3, 2024 5

我是来投票的 烦请移除guzzle/guzzle 请腾讯云的朋友看一下 还有多少使用php5.3 社区的发展与技术的革新需要咱们每个人去推动

from cos-php-sdk-v5.

krwu avatar krwu commented on July 3, 2024 4

@yaozongyou 我还是那个建议,发布一个2.x的版本,改用guzzlehttp。毕竟现在安装的时候都已经警告 guzzle/guzzle is abandoned 了。保留 1.x 的给 php 5.3 的用户就行了。

from cos-php-sdk-v5.

lewzylu avatar lewzylu commented on July 3, 2024 3

1、考虑到很多CVM默认的PHP版本是5.3,如果升级guzzle的话,这部分用户接没法兼容了,所以之前的排期优先级较低。

2、近期我们已经在安排同学来进行相关的重写工作。预计9月初上线

from cos-php-sdk-v5.

overtrue avatar overtrue commented on July 3, 2024 2

已经升级了:https://github.com/tencentyun/cos-php-sdk-v5/blob/master/composer.json#L25

from cos-php-sdk-v5.

andares avatar andares commented on July 3, 2024 1

都过去几百年了,再下去这个库都要没法用了。。更新一个2.0版本使用新库要不了半天时间吧?

from cos-php-sdk-v5.

yveshield avatar yveshield commented on July 3, 2024 1

又过去几百年了,还是没变化

from cos-php-sdk-v5.

HFLeon avatar HFLeon commented on July 3, 2024 1

又过去几百年了,还是没变化

from cos-php-sdk-v5.

allowing avatar allowing commented on July 3, 2024 1

又过去几百年了,还是没变化

from cos-php-sdk-v5.

xiebruce avatar xiebruce commented on July 3, 2024 1

腾讯的人就是这么干事么?这个问题从提出到现在都已经一年零三个月了,还没有解决?技术要严谨,都已经一堆人报这个提示了:
image

我之前还以为我用了老版本,结果找了一遍,本来还以为是小厂,没想到竟然是大厂干的活。。。

都9102年了,老版本的你发个老版本,新版的用新的呀,还有多少人用那老掉牙的。。。

from cos-php-sdk-v5.

tanzhenxing avatar tanzhenxing commented on July 3, 2024

我看到已经更新guzzle 最近版了

from cos-php-sdk-v5.

krwu avatar krwu commented on July 3, 2024
    "require": {
        "php": ">=5.3.0",
        "guzzle/guzzle": "~3.7"
    }

没改变,guzzlehttp/guzzle 是 v4.0 的时候改的名。

from cos-php-sdk-v5.

 avatar commented on July 3, 2024
Version Status Packagist Namespace Repo Docs PSR-7 PHP Version
3.x EOL guzzle/guzzle Guzzle v3 v3 No >= 5.3.3
4.x EOL guzzlehttp/guzzle GuzzleHttp v4 N/A No >= 5.4
5.x Maintained guzzlehttp/guzzle GuzzleHttp v5 v5 No >= 5.4
6.x Latest guzzlehttp/guzzle GuzzleHttp v6 v6 Yes >= 5.5

3.x 版本早已经是 EOL 状态了,但这并不会影响使用这个 SDK(这个版本的架构都是基于老版本的guzzle),使用 3.x 版本可能也是有考虑到兼容低版本的 PHP (>= 5.3.3) #34

from cos-php-sdk-v5.

MikeMaldini avatar MikeMaldini commented on July 3, 2024

那现在怎么说?我是 PHP 7.0 + GUZZLE 6 的 这 SDK 怎么才能正常用

from cos-php-sdk-v5.

 avatar commented on July 3, 2024

@MikeMaldini PHP7.x的环境在composer之后依旧可以使用这个SDK啊

from cos-php-sdk-v5.

MikeMaldini avatar MikeMaldini commented on July 3, 2024

image
image
@vtrois
guzzle 我也装了,没搞明白 我是不是哪里配置有问题

from cos-php-sdk-v5.

krwu avatar krwu commented on July 3, 2024

@MikeMaldini
是 "guzzle/guzzle" 不是 "guzzlehttp/guzzle".

这就是这个 issue 在讨论的问题。

from cos-php-sdk-v5.

MikeMaldini avatar MikeMaldini commented on July 3, 2024

@krwu 所以我必须手动安装老的 guzzle/guzzle

from cos-php-sdk-v5.

slince avatar slince commented on July 3, 2024

@krwu
建议将第三方依赖从版本库移除;使用composer依赖的方式进行管理维护;

from cos-php-sdk-v5.

krwu avatar krwu commented on July 3, 2024

@slince 我不是项目团队的,也是这个项目的使用者。现在本来就是通过composer依赖第三方库。问题在于依赖的guzzle/guzzle版本已经严重过时。

from cos-php-sdk-v5.

slince avatar slince commented on July 3, 2024

@krwu 哦哦

我看了下源代码,项目依赖了guzzle和symfony的一些组件,但都没有写到composer.json 的require里面 这有点可怕了

from cos-php-sdk-v5.

MikeMaldini avatar MikeMaldini commented on July 3, 2024

@slince 我也发现了这个蛋疼的问题

from cos-php-sdk-v5.

krwu avatar krwu commented on July 3, 2024

@slince @MikeMaldini
我也认真看了一遍源码,又用一个空项目做了测试。版本库里的 GuzzleSymfony 目录实际上是给直接下载版本库的人用的。

composer 安装项目的人,直接 require('./vendor/autoload.php') 的情况下,那两个目录没有任何意义。

composer require qcloud/cos-sdk-v5 的时候,会自动安装所需的第三方依赖,并且加载的也是安装的依赖而不是版本库里的 guzzle 和 symfony 相关文件。

测试的步骤:

  1. 新建空文件夹
  2. 在文件夹下执行 composer initcomposer require qcloud/cos-sdk-v5
  3. 删除 vendor/qcloud/cos-sdk-v5/src 下的 GuzzleSymfony 目录
  4. 创建 index.php, 通过 require('./vendor/autoload.php') 自动加载依赖
  5. 最简单的上传测试

执行的情况,测试通过。

from cos-php-sdk-v5.

 avatar commented on July 3, 2024

@yaozongyou 看一下这个问题

from cos-php-sdk-v5.

yaozongyou avatar yaozongyou commented on July 3, 2024

@slince @MikeMaldini 感谢两位的建议:
1、src目录下的guzzle和symfony这两个目录,主要是为了方便不使用composer的用户,能把源码下载下来可以直接使用;
2、使用老版本的guzzle而不是guzzlehttp主要是为了兼容php 5.3的版本;
3、.idea目录稍后将删除;
4、关于单元测试方面,现在有功能性方面的测试代码(src/Qcloud/Cos/Tests/Test.php)。

再次感谢两位的建议,谢谢。

from cos-php-sdk-v5.

yaozongyou avatar yaozongyou commented on July 3, 2024

@krwu 好的,我和@lewzylu先评估下。

from cos-php-sdk-v5.

jjonline avatar jjonline commented on July 3, 2024

底层http请求还是自己实现比较好~

from cos-php-sdk-v5.

xiebruce avatar xiebruce commented on July 3, 2024

@lewzylu 好的👍🏻

from cos-php-sdk-v5.

zuoRambo avatar zuoRambo commented on July 3, 2024

已经升级了:https://github.com/tencentyun/cos-php-sdk-v5/blob/master/composer.json#L25

还是我超哥优秀

from cos-php-sdk-v5.

overtrue avatar overtrue commented on July 3, 2024

@zuoRambo 🤣 不是我改的

from cos-php-sdk-v5.

HFLeon avatar HFLeon commented on July 3, 2024

迟到的GZ

from cos-php-sdk-v5.

lewzylu avatar lewzylu commented on July 3, 2024

v2.0 has relied on guzzle6. if php version <=5.3, please use v1.3

from cos-php-sdk-v5.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.