Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
李少辉-开发者
gitlab-foss
提交
7a56075e
G
gitlab-foss
项目概览
李少辉-开发者
/
gitlab-foss
通知
15
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
G
gitlab-foss
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
7a56075e
编写于
6月 28, 2013
作者:
I
Izaak Alpert
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Some of the requested updates, rebase on master
Change-Id: I305266fe9acbbb5136adeeb52e7e4e1d6629a30a
上级
e86e8818
变更
4
隐藏空白更改
内联
并排
Showing
4 changed file
with
38 addition
and
27 deletion
+38
-27
app/models/user.rb
app/models/user.rb
+8
-0
lib/api/helpers.rb
lib/api/helpers.rb
+8
-15
spec/models/user_spec.rb
spec/models/user_spec.rb
+10
-0
spec/requests/api/api_helpers_spec.rb
spec/requests/api/api_helpers_spec.rb
+12
-12
未找到文件。
app/models/user.rb
浏览文件 @
7a56075e
...
@@ -190,6 +190,14 @@ class User < ActiveRecord::Base
...
@@ -190,6 +190,14 @@ class User < ActiveRecord::Base
def
search
query
def
search
query
where
(
"name LIKE :query OR email LIKE :query OR username LIKE :query"
,
query:
"%
#{
query
}
%"
)
where
(
"name LIKE :query OR email LIKE :query OR username LIKE :query"
,
query:
"%
#{
query
}
%"
)
end
end
def
by_username_or_id
(
name_or_id
)
if
(
name_or_id
.
is_a?
(
Integer
))
User
.
find_by_id
(
name_or_id
)
else
User
.
find_by_username
(
name_or_id
)
end
end
end
end
#
#
...
...
lib/api/helpers.rb
浏览文件 @
7a56075e
...
@@ -12,25 +12,18 @@ module API
...
@@ -12,25 +12,18 @@ module API
if
(
identifier
&&
!
(
@current_user
.
id
==
identifier
||
@current_user
.
username
==
identifier
))
if
(
identifier
&&
!
(
@current_user
.
id
==
identifier
||
@current_user
.
username
==
identifier
))
render_api_error!
(
'403 Forbidden: Must be admin to use sudo'
,
403
)
unless
@current_user
.
is_admin?
render_api_error!
(
'403 Forbidden: Must be admin to use sudo'
,
403
)
unless
@current_user
.
is_admin?
begin
begin
@current_user
=
User
.
by_username_or_id
(
identifier
)
if
(
identifier
.
is_a?
(
Integer
))
user
=
User
.
find_by_id
(
identifier
)
else
user
=
User
.
find_by_username
(
identifier
)
end
if
user
.
nil?
not_found!
(
"No user id or username for:
#{
identifier
}
"
)
end
@current_user
=
user
rescue
=>
ex
rescue
=>
ex
not_found!
(
"No user id or username for:
#{
identifier
}
"
)
not_found!
(
"No user id or username for:
#{
identifier
}
"
)
end
end
not_found!
(
"No user id or username for:
#{
identifier
}
"
)
if
current_user
.
nil?
end
end
@current_user
@current_user
end
end
def
sudo_identifier
()
def
sudo_identifier
()
identifier
=
params
[
SUDO_PARAM
]
==
nil
?
env
[
SUDO_HEADER
]
:
params
[
SUDO_PARAM
]
identifier
||=
params
[
SUDO_PARAM
]
||=
env
[
SUDO_HEADER
]
# Regex for integers
if
(
!!
(
identifier
=~
/^[0-9]+$/
))
if
(
!!
(
identifier
=~
/^[0-9]+$/
))
identifier
.
to_i
identifier
.
to_i
else
else
...
@@ -129,10 +122,10 @@ module API
...
@@ -129,10 +122,10 @@ module API
def
abilities
def
abilities
@abilities
||=
begin
@abilities
||=
begin
abilities
=
Six
.
new
abilities
=
Six
.
new
abilities
<<
Ability
abilities
<<
Ability
abilities
abilities
end
end
end
end
end
end
end
end
spec/models/user_spec.rb
浏览文件 @
7a56075e
...
@@ -208,4 +208,14 @@ describe User do
...
@@ -208,4 +208,14 @@ describe User do
user
.
can_create_group
.
should
==
false
user
.
can_create_group
.
should
==
false
end
end
end
end
describe
'by_username_or_id'
do
let
(
:user1
){
create
(
:user
,
username:
'foo'
)}
it
"should get the correct user"
do
User
.
by_username_or_id
(
user1
.
id
).
should
==
user1
User
.
by_username_or_id
(
'foo'
).
should
==
user1
User
.
by_username_or_id
(
-
1
).
should
be_nil
User
.
by_username_or_id
(
'bar'
).
should
be_nil
end
end
end
end
spec/requests/api/api_helpers_spec.rb
浏览文件 @
7a56075e
require
'spec_helper'
require
'spec_helper'
describe
Gitlab
::
API
do
describe
API
do
include
Gitlab
::
APIHelpers
include
API
::
APIHelpers
include
ApiHelpers
include
ApiHelpers
let
(
:user
)
{
create
(
:user
)
}
let
(
:user
)
{
create
(
:user
)
}
let
(
:admin
)
{
create
(
:admin
)
}
let
(
:admin
)
{
create
(
:admin
)
}
...
@@ -13,27 +13,27 @@ describe Gitlab::API do
...
@@ -13,27 +13,27 @@ describe Gitlab::API do
def
set_env
(
token_usr
,
identifier
)
def
set_env
(
token_usr
,
identifier
)
clear_env
clear_env
clear_param
clear_param
env
[
Gitlab
::
APIHelpers
::
PRIVATE_TOKEN_HEADER
]
=
token_usr
.
private_token
env
[
API
::
APIHelpers
::
PRIVATE_TOKEN_HEADER
]
=
token_usr
.
private_token
env
[
Gitlab
::
APIHelpers
::
SUDO_HEADER
]
=
identifier
env
[
API
::
APIHelpers
::
SUDO_HEADER
]
=
identifier
end
end
def
set_param
(
token_usr
,
identifier
)
def
set_param
(
token_usr
,
identifier
)
clear_env
clear_env
clear_param
clear_param
params
[
Gitlab
::
APIHelpers
::
PRIVATE_TOKEN_PARAM
]
=
token_usr
.
private_token
params
[
API
::
APIHelpers
::
PRIVATE_TOKEN_PARAM
]
=
token_usr
.
private_token
params
[
Gitlab
::
APIHelpers
::
SUDO_PARAM
]
=
identifier
params
[
API
::
APIHelpers
::
SUDO_PARAM
]
=
identifier
end
end
def
clear_env
def
clear_env
env
.
delete
(
Gitlab
::
APIHelpers
::
PRIVATE_TOKEN_HEADER
)
env
.
delete
(
API
::
APIHelpers
::
PRIVATE_TOKEN_HEADER
)
env
.
delete
(
Gitlab
::
APIHelpers
::
SUDO_HEADER
)
env
.
delete
(
API
::
APIHelpers
::
SUDO_HEADER
)
end
end
def
clear_param
def
clear_param
params
.
delete
(
Gitlab
::
APIHelpers
::
PRIVATE_TOKEN_PARAM
)
params
.
delete
(
API
::
APIHelpers
::
PRIVATE_TOKEN_PARAM
)
params
.
delete
(
Gitlab
::
APIHelpers
::
SUDO_PARAM
)
params
.
delete
(
API
::
APIHelpers
::
SUDO_PARAM
)
end
end
def
error!
(
message
,
status
)
def
error!
(
message
,
status
)
...
@@ -42,10 +42,10 @@ describe Gitlab::API do
...
@@ -42,10 +42,10 @@ describe Gitlab::API do
describe
".current_user"
do
describe
".current_user"
do
it
"should leave user as is when sudo not specified"
do
it
"should leave user as is when sudo not specified"
do
env
[
Gitlab
::
APIHelpers
::
PRIVATE_TOKEN_HEADER
]
=
user
.
private_token
env
[
API
::
APIHelpers
::
PRIVATE_TOKEN_HEADER
]
=
user
.
private_token
current_user
.
should
==
user
current_user
.
should
==
user
clear_env
clear_env
params
[
Gitlab
::
APIHelpers
::
PRIVATE_TOKEN_PARAM
]
=
user
.
private_token
params
[
API
::
APIHelpers
::
PRIVATE_TOKEN_PARAM
]
=
user
.
private_token
current_user
.
should
==
user
current_user
.
should
==
user
end
end
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录