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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 2633 天前的主题，其中的信息可能已经有所发展或是发生改变。

几年前就发现他们的 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

技术部的人联系我了，他们说是因为牵扯到一些内部信息，不方便公开。所以我将把代码撤掉~

git

APK

GitHub

65 条回复 • 2017-08-30 14:28:43 +08:00

lusyoe

2017-08-20 20:27:49 +08:00

楼主这样不道德吧，直接就把人家公司的内部产品代码公开出来了~

techmoe

2017-08-20 20:30:25 +08:00

估计再过一会网易的邮件就要到 Livid 或者 lz 的手里了

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

2017-08-20 20:59:11 +08:00

确实不太好

metorm

2017-08-20 21:14:22 +08:00

我很好奇，这个 git 仓库体积是多少呢？

kindjeff

2017-08-20 21:45:43 +08:00

逆向软件是违法的吧～～楼主赶紧细软跑

josephchiang

2017-08-20 22:07:44 +08:00

gh 上 33c5bcd on 12 May 2015 什么鬼我穿越了? ? ? 标题党还是旧闻放送？

RyougiShiki

2017-08-20 22:26:59 +08:00

习惯用 ide 自带的图形界面，比如 pycharm，提交前的清单一目了然，把不需要提交的文件勾去掉。而用命令行的小伙伴偶尔把修改的 settings.py .env 等文件提交上去。

RyougiShiki

2017-08-20 22:29:33 +08:00

为什么外人可以 checkout 下来，他们没设置账号权限吗

pexcn

2017-08-20 23:32:23 +08:00 via Android

@kindjeff @RyougiShiki 这不是逆向，他们把 .git 目录打包进 apk 了…

pexcn

2017-08-20 23:33:02 +08:00 via Android

@josephchiang 这是 2015 年的版本了，所以 commit 是 2015 年的

pexcn

2017-08-20 23:33:40 +08:00 via Android

@metorm 这只是客户端的代码，大概三万多行，而且不包含 resource 的

pexcn

2017-08-20 23:34:19 +08:00 via Android

@haozhang 他们在 3.3.0 版本就已经修复了这个问题，估计 key 早就更新了吧～

pexcn

2017-08-20 23:35:10 +08:00 via Android

@lusyoe 没有…是他们自己公开的，我只是感觉代码不错，发出来，感觉可以学习一个而已…

zhidian

2017-08-20 23:38:28 +08:00

可耻地 fork 了。==

stanjia

2017-08-20 23:39:28 +08:00

网易"公开"课

pexcn

2017-08-20 23:40:13 +08:00 via Android

@stanjia 😂😂😂

kx5d62Jn1J9MjoXP

2017-08-21 00:03:20 +08:00 via Android

真棒，看看大厂到底是个什么水平

odirus

2017-08-21 00:18:23 +08:00

我对 QQ 写成 Qq 这种情况一直比较反感，希望不是我一个人，这样写总感觉怪怪的

Khlieb

2017-08-21 00:20:54 +08:00 via Android

真·网易公开课

Technetiumer

2017-08-21 00:26:49 +08:00 via Android

真 · 网易「公开」课 😂😂😂

lsmgeb89

2017-08-21 01:05:56 +08:00

这违法的吧

maplerecall

2017-08-21 02:14:38 +08:00 via Android

emmm，这版本好早了，不过明天还是给安卓开发那边说下好惹_(:з」∠)_

yyfearth

2017-08-21 05:09:51 +08:00

@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

2017-08-21 05:38:04 +08:00 via Android

@yyfearth 第一种好，因为有的专有名词小写之后我就认不得了，眼生

beginor

2017-08-21 06:18:38 +08:00 via Android

Java 还是太罗嗦，哪位有兴趣翻译成 kotlin ？

tamlok

2017-08-21 07:53:48 +08:00 via Android

@yyfearth 三个字以内的缩写保持大写，以上的按照驼峰

googlebot

2017-08-21 08:06:28 +08:00 via iPad

给 lz 点赞，赶紧 fork

thomaswang

2017-08-21 08:10:29 +08:00 via iPhone

这个包该有多大啊

redtea

2017-08-21 08:46:01 +08:00

JavaBean 如果不注意大小写问题的话，会给自己挖坑。

dong3580

2017-08-21 09:37:26 +08:00

好奇，只靠.git 文件怎么 checkout 源码呢？

hjc4869

2017-08-21 09:46:48 +08:00

@dong3580 直接把 .git 放到一个目录里执行 git reset --hard 就拿到当前 branch 最新的了

BOYPT

2017-08-21 09:47:23 +08:00

(这就是你们 apk 那么大的原因？？

odirus

2017-08-21 09:51:09 +08:00

@yyfearth 嗯，解释得有理有据，已点感谢

BTW，你在 apple 的时候，iOS 都能乱写？要是我的话，我受不鸟

toono

2017-08-21 09:54:10 +08:00

重磅😂

rswl

2017-08-21 09:57:50 +08:00

fork 一下

bkmi

2017-08-21 09:58:08 +08:00 via Android

@pexcn 这怎么不是逆向了，你从安装包里弄出来的，就是逆向

pexcn

2017-08-21 10:00:16 +08:00 via iPhone

@BOYPT 3.2.9 版本的 apk 并不大，5MB 左右而已

21grams

2017-08-21 10:16:00 +08:00

先 fork 了再说

yyfearth

2017-08-21 10:23:31 +08:00 via iPhone

@odirus 显示出来的字符串 iOS 当然必须是对的会有专人翻译和校对
但是代码里面的变量常量类和方法名当然是按照代码规范来写
但是我们不是核心团队所以没有统一的代码规范
我写 JS PHP 的时候

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

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

这些用户是看不见的而且我也不写开放的 API 所以产品和管理不会管的