• 请不要在回答技术问题时复制粘贴 AI 生成的内容
pexcn
V2EX  ›  程序员

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

  •  
  •   pexcn · Aug 20, 2017 · 11197 views
    This topic created in 3365 days ago, the information mentioned may be changed or developed.

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

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

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

    Supplement 1  ·  Aug 21, 2017
    技术部的人联系我了,他们说是因为牵扯到一些内部信息,不方便公开。所以我将把代码撤掉~
    65 replies  •  2017-08-30 14:28:43 +08:00
    lusyoe
    lusyoe
    Aug 20, 2017
    1
    楼主这样不道德吧,直接就把人家公司的内部产品代码公开出来了~
    techmoe
    techmoe
    Aug 20, 2017
    2
    估计再过一会网易的邮件就要到 Livid 或者 lz 的手里了
    haozhang
    haozhang
    Aug 20, 2017
    3
    public static final String QQ_APP_KEY = "801125947";
    public static final String QQ_APP_SECRET = "0cf77f62b7b2288ac72bb38045d3a6b9";
    我觉得网易的人要重新申请 appkey 了。→_→
    mokeyjay
    mokeyjay
    Aug 20, 2017
    4
    确实不太好
    metorm
    metorm
    Aug 20, 2017
    5
    我很好奇,这个 git 仓库体积是多少呢?
    kindjeff
    kindjeff
    Aug 20, 2017
    6
    逆向软件是违法的吧~~楼主赶紧细软跑
    josephchiang
    josephchiang
    Aug 20, 2017
    7
    gh 上 33c5bcd on 12 May 2015 什么鬼我穿越了? ? ? 标题党还是旧闻放送?
    RyougiShiki
    RyougiShiki
    Aug 20, 2017
    8
    习惯用 ide 自带的图形界面,比如 pycharm,提交前的清单一目了然 ,把不需要提交的文件勾去掉。而用命令行的小伙伴偶尔把修改的 settings.py .env 等文件提交上去。
    RyougiShiki
    RyougiShiki
    Aug 20, 2017
    9
    为什么外人可以 checkout 下来,他们没设置账号权限吗
    pexcn
    pexcn
    OP
    Aug 20, 2017 via Android
    10
    @kindjeff @RyougiShiki 这不是逆向,他们把 .git 目录打包进 apk 了…
    pexcn
    pexcn
    OP
    Aug 20, 2017 via Android
    11
    @josephchiang 这是 2015 年的版本了,所以 commit 是 2015 年的
    pexcn
    pexcn
    OP
    Aug 20, 2017 via Android
    12
    @metorm 这只是客户端的代码,大概三万多行,而且不包含 resource 的
    pexcn
    pexcn
    OP
    Aug 20, 2017 via Android
    13
    @haozhang 他们在 3.3.0 版本就已经修复了这个问题,估计 key 早就更新了吧~
    pexcn
    pexcn
    OP
    Aug 20, 2017 via Android
    14
    @lusyoe 没有…是他们自己公开的,我只是感觉代码不错,发出来,感觉可以学习一个而已…
    zhidian
    zhidian
    Aug 20, 2017
    15
    可耻地 fork 了。==
    stanjia
    stanjia
    Aug 20, 2017❤️2
    16
    网易"公开"课
    pexcn
    pexcn
    OP
    Aug 20, 2017 via Android
    17
    @stanjia 😂😂😂
    kx5d62Jn1J9MjoXP
    kx5d62Jn1J9MjoXP
    Aug 21, 2017 via Android
    18
    真棒,看看大厂到底是个什么水平
    odirus
    odirus
    Aug 21, 2017
    19
    我对 QQ 写成 Qq 这种情况一直比较反感,希望不是我一个人,这样写总感觉怪怪的
    Khlieb
    Khlieb
    Aug 21, 2017 via Android
    20
    真·网易公开课
    Technetiumer
    Technetiumer
    Aug 21, 2017 via Android
    21
    真 · 网易「公开」课 😂😂😂
    lsmgeb89
    lsmgeb89
    Aug 21, 2017
    22
    这违法的吧
    maplerecall
    maplerecall
    Aug 21, 2017 via Android
    23
    emmm,这版本好早了,不过明天还是给安卓开发那边说下好惹_(:з」∠)_
    yyfearth
    yyfearth
    Aug 21, 2017❤️3
    24
    @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
    binux
    Aug 21, 2017 via Android
    25
    @yyfearth 第一种好,因为有的专有名词小写之后我就认不得了,眼生
    beginor
    beginor
    Aug 21, 2017 via Android
    26
    Java 还是太罗嗦, 哪位有兴趣翻译成 kotlin ?
    tamlok
    tamlok
    Aug 21, 2017 via Android
    27
    @yyfearth 三个字以内的缩写保持大写,以上的按照驼峰
    googlebot
    googlebot
    Aug 21, 2017 via iPad
    28
    给 lz 点赞,赶紧 fork
    thomaswang
    thomaswang
    Aug 21, 2017 via iPhone
    29
    这个包该有多大啊
    redtea
    redtea
    Aug 21, 2017
    30
    JavaBean 如果不注意大小写问题的话,会给自己挖坑。
    dong3580
    dong3580
    Aug 21, 2017
    31
    好奇,只靠.git 文件怎么 checkout 源码呢?
    hjc4869
    hjc4869
    Aug 21, 2017❤️3
    32
    @dong3580 直接把 .git 放到一个目录里执行 git reset --hard 就拿到当前 branch 最新的了
    BOYPT
    BOYPT
    Aug 21, 2017
    33
    (这就是你们 apk 那么大的原因??
    odirus
    odirus
    Aug 21, 2017
    34
    @yyfearth 嗯,解释得有理有据,已点感谢

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

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

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

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















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

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

    is_iOS_IPSec_tunnel_ready_for_SaaS_API
    get_QQ_FQDN_IP_in_JSON

    同专有名词写小写不太认得出来。
    yukiww233
    yukiww233
    Aug 21, 2017
    60
    opencourse 和 opensource 很接近啊
    没准人家是故意的呢 233333
    kalsir
    kalsir
    Aug 21, 2017
    61
    刚看到了,然后刷新了一下,没了。。
    pexcn
    pexcn
    OP
    Aug 21, 2017
    62
    @Gothack 然而有很多 fork 的。。。( Run...
    Trumeet
    Trumeet
    Aug 21, 2017
    63
    來晚了,43 分鐘前 Removed (
    pexcn
    pexcn
    OP
    Aug 21, 2017 via iPhone
    64
    @Trumeet 😂不是有很多 fork 嘛。。
    llxx510200
    llxx510200
    Aug 30, 2017
    65
    @laoyur 还能发我一份吗,,大佬
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   2433 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 166ms · UTC 11:06 · PVG 19:06 · LAX 03:06 · JFK 06:06
    ♥ Do have faith in what you're doing.