1. 10 6月, 2016 14 次提交
  2. 09 6月, 2016 26 次提交
    • D
      Merge branch 'fix-git-http-routing' into 'master' · 3803b380
      Douwe Maan 提交于
      Ensure only IDs ending in .git perform git actions
      
      ## What does this MR do?
      
      Rails's routing is pretty strange. Previously, `GET /namespace/project/info/refs` would go to the Git HTTP controller (if the redirect for that case was taken out).
      
      ## Are there points in the code the reviewer needs to double check?
      
      The specs fail if the redirect is moved to above the Git HTTP routes, removed altogether, or the Git HTTP constraints are changed. But there might still be missing cases.
      
      ## Why was this MR needed?
      
      The master build and HTTP cloning were both broken.
      
      ## What are the relevant issue numbers?
      
      Closes #18376.
      
      ## Screenshots (if relevant)
      
      Nope.
      
      ## Does this MR meet the acceptance criteria?
      
      - [x] [not needed] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added
      - [x] [not needed] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)
      - [ ] (not needed) API support added
      - [ ] Tests
        - [x] Added for this feature/bug
        - [ ] All builds are passing
      - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
      - [ ] Branch has no merge conflicts with `master` (if you do - rebase it please)
      - [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
      
      See merge request !4558
      3803b380
    • S
      Add test for getting info/refs from repo · bf63964b
      Sean McGivern 提交于
      bf63964b
    • S
      Ensure only IDs ending in .git perform git actions · df5fb28a
      Sean McGivern 提交于
      It doesn't seem possible to set constraints based on format for project
      IDs ending in .git, so set the constraint on the ID and ensure the
      format is nil to avoid the case where the project ID is something like
      project.git.foo.
      df5fb28a
    • D
      Merge branch 'saml-ldap-link-flow' into 'master' · 30ee4ea6
      Douwe Maan 提交于
      Adjust the SAML control flow to allow LDAP identities to be added to an existing SAML user.
      
      ## What does this MR do?
      
      It correctly lets an existing SAML user to add their LDAP identity automatically at login.
      
      ## Why was this MR needed?
      
      A customer had issues with the `auto_link_ldap_user` feature. The flow was not working if there was an account with a SAML identity, but no LDAP identity. GitLab would pick up the correct LDAP person, but due to the order of the flow, that LDAP person was never associated with the user.
      
      ## What are the relevant issue numbers?
      
      Fixes #17346 
      
      /cc @dblessing @balameb @stanhu 
      
      See merge request !4498
      30ee4ea6
    • Y
      Merge branch 'make-omniauth-tests-to-not-modify-global-state' into 'master' · 1e4db9ed
      Yorick Peterse 提交于
      Make Omniauth providers specs to not modify global configuration
      
      See merge request !4534
      1e4db9ed
    • R
      Merge branch '18019-fix-wiki-linking' into 'master' · 3d72cb89
      Rémy Coutable 提交于
      Fix wiki linking behavior for markdown wiki pages
      
      Related to #18019 
      
      - As per the documentation in !4372 
      
      ## TODO
      - [ ] !4432 Have wiki linking behave as per the documentation
          - [x] Move `WikiLinkFilter` specs to the pipeline level
          - [x] Verify current behavior on wiki `show` page
          - [x] Fix current behavior on wiki `show` page
          - [x] Verify current behaviour on wiki preview
          - [x] Fix current behaviour on wiki preview
              - [x] Rewrite all links and get preview links working
              - [x] Make sure all links are on-par with the wiki `show` page
              - [x] TDD `WikiLinkFilter` and get it working
              - [x] Hook `WikiLinkFilter` up
          - [x] Fix tests 
              - [x] Fix `markdown_spec`
              - [x] Fix `wiki` spinach feature
              - [x] Wait for [build](https://gitlab.com/gitlab-org/gitlab-ce/commit/4f50dd2/builds) to pass
              - [x] Make sure all wiki-related pages are working as expected (history, all pages, etc.)
          - [x] Test in different ruby versions
          - [x] GitLab instances hosted on a relative URL
          - [x] Non-markdown rendering formats?
              - [x] RDoc
              - [x] ASCIIDoc
              - [x] Create issues to fix things for RDoc and ASCIIDoc
          - [x] Gauge performance impact
          - [x] Refactor
          - [x] Re-organize commits
          - [x] Make sure [build](https://gitlab.com/gitlab-org/gitlab-ce/commit/f860e9a8dcabe7d5f160c32fc549807c98baa4a1/builds) passes
          - [x] Respond to @rymai's comments
              - [x] `class WikiLinkFilter < HTML::Pipeline::Filter`
              - [x] blank line after guard clause
              - [x] keyword arguments for `wiki` and `slug`
              - [x] invert the condition
              - [x] inline `user` in spec
              - [x] Make sure spec names are not out of date
              - [x] Comment for each rewrite rule
          - [x] Add CHANGELOG entry
          - [x] Reorganize commits
          - [x] Make sure [build](https://gitlab.com/gitlab-org/gitlab-ce/commit/19b91e749a6320d12fb299d33f1f6440777e0e26/builds) passes
          - [ ] Wait for merge
      
      See merge request !4432
      3d72cb89
    • T
      Add #18019 to the CHANGELOG. · 19b91e74
      Timothy Andrew 提交于
      19b91e74
    • T
      Hook up the updated `WikiLinkFilter` to the wiki controllers. · e6b1d166
      Timothy Andrew 提交于
      - Need to pass in a `page_slug` to the filter, so it can rewrite based
        on the current page (all links are rewritten to the level of the app root).
      - The earlier `markdown_preview` endpoint was at the level of the wiki.
        We need to know the current page (for rewriting, as above), so this
        commit moves the endpoint to the level of a wiki page.
      - Fix all tests
      e6b1d166
    • T
      Implement the correct linking behaviour in `WikiLinkFilter`. · 8e71c19a
      Timothy Andrew 提交于
      Original Comments
      =================
      
      - Linking behaves as per rules documented here:
        https://gitlab.com/gitlab-org/gitlab-ce/blob/16568-document-wiki-linking-behavior/doc/markdown/wiki.md
      - All links (to other wiki pages) are rewritten to be at the level of
        the app root. We can't use links relative to the current
        page ('./foo', 'foo', '../foo'), because they won't work in the
        markdown preview, where the current page is suffixed with `/edit`
      - Move existing `WikiLinkFilter` specs to `WikiPipeline` spec. It makes
        sense to run these tests on the combined output of the pipeline,
        rather than a single filter, since we can catch issues with
        conflicting filters.
      - Add more tests to cover the new linking
      
      @rymai's Review
      ===============
      
      - Classes nested under `WikiLinkFilter` should declare `WikiLinkFilter`'s
        inherit, so nothing changes if the nested class is loaded first.
      - Add a blank line after a guard clause
      - Use keyword arguments for the `Rewriter` constructor
      - Invert a condition - use `if` instead of `unless`
      - Inline a `let` in `WikiPipeline` spec - it was only used in a single place
      - Change out of date spec names
      - Add a comment for every rewrite rule in `Rewriter`
      8e71c19a
    • J
      Merge branch 'fix-award-emoji-hidden' into 'master' · 13fd88fa
      Jacob Schatz 提交于
      Shows award emoji for comments to all users who are logged in.
      
      ## What does this MR do?
      Fixes the award emoji picker not showing up for users who weren't able to edit another user's comment.
      
      ## Screenshots (if relevant)
      Before:
      
      ![Screen_Shot_2016-06-08_at_5.18.04_PM](/uploads/a8b40ba319481f53d55de055d84649b9/Screen_Shot_2016-06-08_at_5.18.04_PM.png)
      
      After (logged in):
      ![Screen_Shot_2016-06-08_at_5.16.16_PM](/uploads/535edcca6fbdd8834beda39399f85384/Screen_Shot_2016-06-08_at_5.16.16_PM.png)
      
      After (logged out):
      ![Screen_Shot_2016-06-08_at_5.15.08_PM](/uploads/7f0dff751714c7d20f78b56893cacd59/Screen_Shot_2016-06-08_at_5.15.08_PM.png)
      
      cc: @fatihacet 
      
      See merge request !4547
      13fd88fa
    • J
      Merge branch 'mention-loading-indicator' into 'master' · f5818baf
      Jacob Schatz 提交于
      Show loading indicator for autocomplete
      
      ![autocomplete](/uploads/397df34689aeb5f4cbca99aea209c8cf/autocomplete.gif)
      
      Closes #15435
      
      See merge request !4182
      f5818baf
    • J
      Merge branch '17284-Edit-form-does-not-cancel-correctly' into 'master' · e382f130
      Jacob Schatz 提交于
      #17284 Edit form does not cancel correctly
      
      ## What does this MR do?
      
      Replaces the markdown textarea with the currently saved note text when the note edit is cancelled.
      
      ## Are there points in the code the reviewer needs to double check?
      
      No
      
      ## Why was this MR needed?
      
      Allows the user to revert back to the original note text, as expected, using cancel.
      
      ## What are the relevant issue numbers?
      
      Fixes #17284.
      
      ## Screenshots
      
      ![17284](/uploads/c3598de6f8e70d52275ef25907d92d42/17284.gif)
      
      See merge request !4175
      e382f130
    • J
      Merge branch 'nav-scroll-bug' into 'master' · cc54b74a
      Jacob Schatz 提交于
      Side nav scrolls if content height taller than screen
      
      ## What does this MR do?
      Header logo is now fixed at top; put back scrolling functionality in side nav
      
      ## Screenshots (if relevant)
      <img src="/uploads/e1d1b60c296c39c61d7fe7da22b6c52d/scroll.gif" width="600px">
      
      See merge request !4513
      cc54b74a
    • R
      Merge branch 'git-http-no-authenticity-token' into 'master' · 5d66e0e0
      Robert Speicher 提交于
      Disable CSRF protection when serving Git HTTP clients
      
      Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/18331
      
      See merge request !4538
      5d66e0e0
    • C
      bdc995bf
    • P
      9282810f
    • J
      Merge branch '17240-svg' into 'master' · 0001a0cf
      Jacob Schatz 提交于
      Add new icons in left sidebar and layout nav
      
      Closes #17609    
      Closes #17240    
      Closes #17321    
      
      ![Screen_Shot_2016-06-02_at_8.47.38_AM](/uploads/f254fe81cba65cbdeb8e5720f0b2fc19/Screen_Shot_2016-06-02_at_8.47.38_AM.png)
      ![Screen_Shot_2016-06-02_at_8.53.33_AM](/uploads/2c27284f0a4916fe447794d9b663c4df/Screen_Shot_2016-06-02_at_8.53.33_AM.png)
      
      @skyruler I couldn't find the icons for `Projects`, `Merge Requests`, `Snippets`, or `Help` in the design repo. Do you have them available somewhere?
      
      See merge request !4268
      0001a0cf
    • J
      Merge branch '18333-layout-bug' into 'master' · 8578f401
      Jacob Schatz 提交于
      Use container helper instead of hard coded container
      
      ## What does this MR do?
      Add `container_class` helper class to fix layout bug
      
      ## What are the relevant issue numbers?
      #18333 
      ## Screenshots (if relevant)
      Fixed:    
      ![Screen_Shot_2016-06-08_at_2.03.38_PM](/uploads/6e7575d711efb8eaf6ba641907b2d607/Screen_Shot_2016-06-08_at_2.03.38_PM.png)
      Fluid:     
      ![Screen_Shot_2016-06-08_at_2.03.56_PM](/uploads/33d3f73781ca9ed129f853abfe9feb4b/Screen_Shot_2016-06-08_at_2.03.56_PM.png)
      
      See merge request !4543
      8578f401
    • A
      Fix activity SVG · a280116a
      Annabel Dunstone 提交于
      a280116a
    • J
      Merge branch 'dropdown-link-lineheight' into 'master' · b4812e2e
      Jacob Schatz 提交于
      Fixed dropdown link line-height
      
      ## What does this MR do?
      
      Fixes line-height issue on dropdown menu links
      
      ## Why was this MR needed?
      
      The line-height of some dropdown links were taking the parent, most noticeable the commit header
      
      ## Screenshots (if relevant)
      
      Before:
      
      ![Screen_Shot_2016-06-08_at_12.26.13](/uploads/51d6bbb4d727da0e3f27d715d6556d4f/Screen_Shot_2016-06-08_at_12.26.13.png)
      
      After:
      
      ![Screen_Shot_2016-06-08_at_12.25.03](/uploads/2dcbfaa67982500eef42cfe4247cd54d/Screen_Shot_2016-06-08_at_12.25.03.png)
      
      See merge request !4531
      b4812e2e
    • J
      Merge branch 'dz-improve-ui-consistency' into 'master' · 2268c6c7
      Jacob Schatz 提交于
      Improve ui consistency
      
      * Decline idea of using gray buttons actively in UI
      * remove icons from some buttons that already has text
      * standardizes margin between buttons 
      
      
      First step in https://gitlab.com/gitlab-org/gitlab-ce/issues/18324 
      
      See merge request !4533
      2268c6c7
    • R
      Merge branch '14098-customize-after-sign-up-path' into 'master' · 18f5cca2
      Robert Speicher 提交于
      Allow custom text on 'almost there' page
      
      Add a new application setting, after_sign_up_text. This is text to be
      rendered as Markdown and shown on the 'almost there' page after a user
      signs up, but before they've confirmed their account.
      
      Closes #14098.
      
      See merge request !4098
      18f5cca2
    • R
      Merge branch '13840-allow-clones-from-http-url-s-without-appending-git' into 'master' · fe75d163
      Robert Speicher 提交于
      Allow clones from /namespace/project
      
      Allow `git clone https://host/namespace/project` to work, in addition to `git clone https://host/namespace/project.git`
      
      Closes #13840.
      
      See merge request !4530
      fe75d163
    • J
      Skip authenticity token checks for Git HTTP · 63ed8032
      Jacob Vosmaer 提交于
      63ed8032
    • S
      Allow clones from /namespace/project · cba32b71
      Sean McGivern 提交于
      Redirect /namespace/project/info/refs to
      /namespace/project.git/info/refs, so `git clone` works on either
      namespace/project.git or namespace/project.
      cba32b71
    • R
      Merge branch 'password-min-length-placeholder' into 'master' · d74e93ce
      Robert Speicher 提交于
      Added minimum password length to password field
      
      Add a placeholder to the password field in the register form with the minimum number of characters required. Also added a pattern to the field to stop it submitting if less than that.
      
      Closes #17765
      
      See merge request !4458
      d74e93ce