V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
pexcn
V2EX  ›  程序员

发现网易公开课 Android 版不小心在 release apk 的时候把 .git 目录打包了进去

  •  
  •   pexcn · 2017-08-20 17:57:57 +08:00 · 9491 次点击
    这是一个创建于 2412 天前的主题,其中的信息可能已经有所发展或是发生改变。

    几年前就发现他们的 apk 里面有个 .git 目录,今天闲着没事就拿出来翻一翻。 然后发现从不知道那个版本开始,一直到 3.2.9 版,apk 里面都有个 .git 目录。

    然后我就把它的 source checkout 了出来,看了看,感觉写得不错。 只是 code style 有待改进,可以学习一个~

    顺手放到了 GitHub: https://github.com/pexcn/com.netease.vopen

    第 1 条附言  ·  2017-08-21 18:35:42 +08:00
    技术部的人联系我了,他们说是因为牵扯到一些内部信息,不方便公开。所以我将把代码撤掉~
    65 条回复    2017-08-30 14:28:43 +08:00
    lusyoe
        1
    lusyoe  
       2017-08-20 20:27:49 +08:00
    楼主这样不道德吧,直接就把人家公司的内部产品代码公开出来了~
    techmoe
        2
    techmoe  
       2017-08-20 20:30:25 +08:00
    估计再过一会网易的邮件就要到 Livid 或者 lz 的手里了
    haozhang
        3
    haozhang  
       2017-08-20 20:58:20 +08:00
    public static final String QQ_APP_KEY = "801125947";
    public static final String QQ_APP_SECRET = "0cf77f62b7b2288ac72bb38045d3a6b9";
    我觉得网易的人要重新申请 appkey 了。→_→
    mokeyjay
        4
    mokeyjay  
       2017-08-20 20:59:11 +08:00
    确实不太好
    metorm
        5
    metorm  
       2017-08-20 21:14:22 +08:00
    我很好奇,这个 git 仓库体积是多少呢?
    kindjeff
        6
    kindjeff  
       2017-08-20 21:45:43 +08:00
    逆向软件是违法的吧~~楼主赶紧细软跑
    josephchiang
        7
    josephchiang  
       2017-08-20 22:07:44 +08:00
    gh 上 33c5bcd on 12 May 2015 什么鬼我穿越了? ? ? 标题党还是旧闻放送?
    RyougiShiki
        8
    RyougiShiki  
       2017-08-20 22:26:59 +08:00
    习惯用 ide 自带的图形界面,比如 pycharm,提交前的清单一目了然 ,把不需要提交的文件勾去掉。而用命令行的小伙伴偶尔把修改的 settings.py .env 等文件提交上去。
    RyougiShiki
        9
    RyougiShiki  
       2017-08-20 22:29:33 +08:00
    为什么外人可以 checkout 下来,他们没设置账号权限吗
    pexcn
        10
    pexcn  
    OP
       2017-08-20 23:32:23 +08:00 via Android
    @kindjeff @RyougiShiki 这不是逆向,他们把 .git 目录打包进 apk 了…
    pexcn
        11
    pexcn  
    OP
       2017-08-20 23:33:02 +08:00 via Android
    @josephchiang 这是 2015 年的版本了,所以 commit 是 2015 年的
    pexcn
        12
    pexcn  
    OP
       2017-08-20 23:33:40 +08:00 via Android
    @metorm 这只是客户端的代码,大概三万多行,而且不包含 resource 的
    pexcn
        13
    pexcn  
    OP
       2017-08-20 23:34:19 +08:00 via Android
    @haozhang 他们在 3.3.0 版本就已经修复了这个问题,估计 key 早就更新了吧~
    pexcn
        14
    pexcn  
    OP
       2017-08-20 23:35:10 +08:00 via Android
    @lusyoe 没有…是他们自己公开的,我只是感觉代码不错,发出来,感觉可以学习一个而已…
    zhidian
        15
    zhidian  
       2017-08-20 23:38:28 +08:00
    可耻地 fork 了。==
    stanjia
        16
    stanjia  
       2017-08-20 23:39:28 +08:00   ❤️ 2
    网易"公开"课
    pexcn
        17
    pexcn  
    OP
       2017-08-20 23:40:13 +08:00 via Android
    @stanjia 😂😂😂
    ssynhtn
        18
    ssynhtn  
       2017-08-21 00:03:20 +08:00 via Android
    真棒,看看大厂到底是个什么水平
    odirus
        19
    odirus  
       2017-08-21 00:18:23 +08:00
    我对 QQ 写成 Qq 这种情况一直比较反感,希望不是我一个人,这样写总感觉怪怪的
    Khlieb
        20
    Khlieb  
       2017-08-21 00:20:54 +08:00 via Android
    真·网易公开课
    Technetiumer
        21
    Technetiumer  
       2017-08-21 00:26:49 +08:00 via Android
    真 · 网易「公开」课 😂😂😂
    lsmgeb89
        22
    lsmgeb89  
       2017-08-21 01:05:56 +08:00
    这违法的吧
    maplerecall
        23
    maplerecall  
       2017-08-21 02:14:38 +08:00 via Android
    emmm,这版本好早了,不过明天还是给安卓开发那边说下好惹_(:з」∠)_
    yyfearth
        24
    yyfearth  
       2017-08-21 05:09:51 +08:00   ❤️ 3
    @odirus 很合理的编码规范啊 严格遵守驼峰 就算是专有名词 我记得貌似 Google 的规范就是这样
    你想想那么多专有名词 如果保持原样写在一起根本分不出来
    QQ IP 之类的还好 iOS iPhone macOS 就更糟糕了

    以前在 Apple 的时候 我们小组没有统一编码规范 有人坚持要保证这些专有名词的大小写正确
    而我和几个新来的又坚持严格遵守驼峰 于是 iOS IOS Ios 混杂 代码没法看了

    看看下面的这些方法 你觉得是那个好呢?

    is iOS IPSec Tunnel Ready for SaaS API:
    isiOSIPSecTunnelReadyforSaaSAPI
    isIosIpsecTunnelReadyForSaasApi

    get QQ FQDN IP in JSON:
    getQQFQDNIPinJSON
    getQqFqdnIpInJson
    binux
        25
    binux  
       2017-08-21 05:38:04 +08:00 via Android
    @yyfearth 第一种好,因为有的专有名词小写之后我就认不得了,眼生
    beginor
        26
    beginor  
       2017-08-21 06:18:38 +08:00 via Android
    Java 还是太罗嗦, 哪位有兴趣翻译成 kotlin ?
    tamlok
        27
    tamlok  
       2017-08-21 07:53:48 +08:00 via Android
    @yyfearth 三个字以内的缩写保持大写,以上的按照驼峰
    googlebot
        28
    googlebot  
       2017-08-21 08:06:28 +08:00 via iPad
    给 lz 点赞,赶紧 fork
    thomaswang
        29
    thomaswang  
       2017-08-21 08:10:29 +08:00 via iPhone
    这个包该有多大啊
    redtea
        30
    redtea  
       2017-08-21 08:46:01 +08:00
    JavaBean 如果不注意大小写问题的话,会给自己挖坑。
    dong3580
        31
    dong3580  
       2017-08-21 09:37:26 +08:00
    好奇,只靠.git 文件怎么 checkout 源码呢?
    hjc4869
        32
    hjc4869  
       2017-08-21 09:46:48 +08:00   ❤️ 3
    @dong3580 直接把 .git 放到一个目录里执行 git reset --hard 就拿到当前 branch 最新的了
    BOYPT
        33
    BOYPT  
       2017-08-21 09:47:23 +08:00
    (这就是你们 apk 那么大的原因??
    odirus
        34
    odirus  
       2017-08-21 09:51:09 +08:00
    @yyfearth 嗯,解释得有理有据,已点感谢

    BTW,你在 apple 的时候,iOS 都能乱写?要是我的话,我受不鸟
    toono
        35
    toono  
       2017-08-21 09:54:10 +08:00
    重磅😂
    rswl
        36
    rswl  
       2017-08-21 09:57:50 +08:00
    fork 一下
    bkmi
        37
    bkmi  
       2017-08-21 09:58:08 +08:00 via Android
    @pexcn 这怎么不是逆向了,你从安装包里弄出来的,就是逆向
    pexcn
        38
    pexcn  
    OP
       2017-08-21 10:00:16 +08:00 via iPhone
    @BOYPT 3.2.9 版本的 apk 并不大,5MB 左右而已
    21grams
        39
    21grams  
       2017-08-21 10:16:00 +08:00
    先 fork 了再说
    yyfearth
        40
    yyfearth  
       2017-08-21 10:23:31 +08:00 via iPhone   ❤️ 1
    @odirus 显示出来的字符串 iOS 当然必须是对的 会有专人翻译和校对
    但是代码里面的变量 常量 类和方法名 当然是按照代码规范来写
    但是我们不是核心团队所以没有统一的代码规范
    我写 JS PHP 的时候

    常量全大写 IOS_VERSION 总不会写成 iOS_VERSON 吧

    其它情况就很多了 有 IOSClass IosClass IPhoneClass IphoneClass 甚至 iOSClass iPhoneClass 的
    getIOSVersion getIosVersion 甚至 getiOSVersion 的

    这些用户是看不见的 而且我也不写开放的 API 所以产品和管理不会管的
    laoyur
        41
    laoyur  
       2017-08-21 10:23:34 +08:00
    说先 fork 为敬的人,你们也太...















    naive 了,一旦被 DMCA,你们的 fork 还能留?又不是没有先例!
    所以,我直接 download zip😝
    bk201
        42
    bk201  
       2017-08-21 10:23:51 +08:00
    这实习生写的吧
    odirus
        43
    odirus  
       2017-08-21 10:25:01 +08:00
    @yyfearth 好的,感谢解惑
    pexcn
        44
    pexcn  
    OP
       2017-08-21 10:30:43 +08:00
    @laoyur 其实就算被 DMCA, 也可以从它的 3.2.9 版的 .git 目录里再 checkout 出来的...
    corvofeng
        45
    corvofeng  
       2017-08-21 10:37:33 +08:00 via Android
    真的是可耻的 fork 了, 也是学习一下吧
    zzczzc
        46
    zzczzc  
       2017-08-21 10:37:50 +08:00
    看一下大产的水平
    VYSE
        47
    VYSE  
       2017-08-21 10:42:44 +08:00
    说不定故意开源呢,赞一个支持开源
    FanWall
        48
    FanWall  
       2017-08-21 10:46:26 +08:00 via Android
    网易大概会怎么应对?
    pexcn
        49
    pexcn  
    OP
       2017-08-21 10:47:00 +08:00
    @FanWall 应该无所谓了,毕竟是两年前的代码了
    yefuchao
        50
    yefuchao  
       2017-08-21 11:12:52 +08:00
    可耻的 Git clone 到本地了= P
    zhuangtongfa
        51
    zhuangtongfa  
       2017-08-21 11:18:46 +08:00
    几年前的代码了
    devzero
        52
    devzero  
       2017-08-21 11:35:14 +08:00
    从 3.0.1 到 3.2.9,中间 14 个版本都有.git 目录
    voocel
        53
    voocel  
       2017-08-21 11:50:43 +08:00
    网易这开源精神值得赞扬
    Chyroc
        54
    Chyroc  
       2017-08-21 12:31:16 +08:00
    把 git 目录放到 git,那么不是永远有更改的代码?

    刚 commit,git 目录又改了
    maplerecall
        55
    maplerecall  
       2017-08-21 12:40:47 +08:00 via Android
    嗯……目前评估影响不大的样子,里边东西都挺旧了,你们就把这当做一节公开课好咯 XD
    Kalv8n
        56
    Kalv8n  
       2017-08-21 12:48:53 +08:00 via Android
    强行开源
    acoder2013
        57
    acoder2013  
       2017-08-21 12:55:21 +08:00
    在课程详情中,增加北京广告信息 。。。
    Hzzone
        58
    Hzzone  
       2017-08-21 12:58:17 +08:00
    宝宝赶紧 fork 了一份...替楼主烧香。。。
    lilydjwg
        59
    lilydjwg  
       2017-08-21 13:14:20 +08:00
    @yyfearth #24

    is_iOS_IPSec_tunnel_ready_for_SaaS_API
    get_QQ_FQDN_IP_in_JSON

    同专有名词写小写不太认得出来。
    yukiww233
        60
    yukiww233  
       2017-08-21 13:39:20 +08:00
    opencourse 和 opensource 很接近啊
    没准人家是故意的呢 233333
    Gothack
        61
    Gothack  
       2017-08-21 18:38:18 +08:00
    刚看到了,然后刷新了一下,没了。。
    pexcn
        62
    pexcn  
    OP
       2017-08-21 18:40:02 +08:00
    @Gothack 然而有很多 fork 的。。。( Run...
    Trumeet
        63
    Trumeet  
       2017-08-21 19:20:38 +08:00
    來晚了,43 分鐘前 Removed (
    pexcn
        64
    pexcn  
    OP
       2017-08-21 19:22:17 +08:00 via iPhone
    @Trumeet 😂不是有很多 fork 嘛。。
    llxx510200
        65
    llxx510200  
       2017-08-30 14:28:43 +08:00
    @laoyur 还能发我一份吗,,大佬
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   1635 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 33ms · UTC 16:51 · PVG 00:51 · LAX 09:51 · JFK 12:51
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.