mapbox / mapbox/assert-http

handlers inside of handlers

Đang mở
#16 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Ngôn ngữ chính
JavaScript
Star
5
Fork
8
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

If we could nest handers inside of handlers

{hash_recursive_make:a_make:b}

handler['recursive'] = function(req, optStr, next) {
  var opts = optStr.split('_');

  for(var i=0; i<opts.length; i++) {
    if(opts[i].indexOf('make:') === 0) {
      var username = opts[i].replace('make:', '');
      return makeUser(username, function(err, userid) {
        if(err) next(err);
        opts[i] = userid;
        return handler['recursive'](req, opts.join('_'), next);
      }
    }
  }

  return hashOf(opts);
}

would be

{hash_{user_a}_{user_b}}

hander['hash'] = function(req, optsStr, next) {
   return hashOf(optsStr.split('_'));
}

hander['user'] = function(req, username, next) {
   makeUser(username, next);
}

Does this seem like something we want to support? If so, I can create a PR to make it happen.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Không có tệp hoặc test nào được nêu tên. Hãy bắt đầu bằng cách xác định entry point hiện có để parsing và dispatch handler, sau đó so sánh hành vi của nó với cú pháp lồng nhau được đề xuất {hash_{user_a}_{user_b}}. Công việc được hoàn tất khi có một thiết kế đã thống nhất và các test chứng minh các handler lồng nhau.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
javascript
Lĩnh vực
testing
Loại issue
Tính năng
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
25/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.