Project

General

Profile

Actions

User story #3751

closed

User story #3000: Build a REST API to manage all actions in Rudder (CRUD on Nodes, Groups, Directives and Rules)

Allow API v1 URL on localhost only

Added by François ARMAND over 10 years ago. Updated almost 9 years ago.

Status:
Released
Priority:
2
Assignee:
Matthieu CERDA
Category:
API
Target version:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

On Rudder 2.6, we could request "/api/*" without any authentication, and so we choosed to allows URL of that kind to be called from localhost only.

In Rudder 2.7, we are going to get a real API with working authentication, but we need to still allows non authenticated request on URL from API v1.

So, for the following URL ONLY, Apache have to be configured to accept request from localhost only
(has it was, but in place of "/api/*", we take care of):

- /api/status
- /api/techniqueLibrary/reload
- /api/dyngroup/reload
- /api/deploy/reload
- /api/archives

Actions #1

Updated by François ARMAND over 10 years ago

  • Status changed from 8 to Pending release
  • % Done changed from 0 to 100
Actions #3

Updated by François ARMAND over 10 years ago

  • Status changed from Pending release to 12

Bad commit message

Actions #4

Updated by Matthieu CERDA over 10 years ago

  • % Done changed from 100 to 0

Fixing the ticket status, as the commit message is wrong and does not concern this ticket.

Actions #5

Updated by Matthieu CERDA over 10 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100
  • Pull Request set to https://github.com/Normation/rudder/pull/265
Actions #6

Updated by Nicolas PERRON over 10 years ago

The commit f5d75ce4 was for #3745, for now, this issue has no commit associated

Actions #7

Updated by Jonathan CLARKE over 10 years ago

Nicolas PERRON wrote:

The commit f5d75ce4 was for #3745, for now, this issue has no commit associated

So please edit it in Redmine to reflect this (click on the commit and you can add/remove linked issues)

Actions #8

Updated by Matthieu CERDA over 10 years ago

  • Status changed from Pending technical review to Pending release
Actions #10

Updated by Matthieu CERDA over 10 years ago

The commits that have been wrongly associated with this ticket were moved to the right one, everything is in order here.

Actions #11

Updated by Nicolas PERRON over 10 years ago

  • Status changed from Pending release to Released
Actions #12

Updated by Benoît PECCATTE almost 9 years ago

  • Tracker changed from 6 to User story
Actions

Also available in: Atom PDF