Sometimes namespace gets stuck in "Activating" state.

Rancher versions: Build from master
Steps to Reproduce:
Had an imported RKE cluster.
Created a new project.
Created a workload in a new namespace in this project.
namespace gets stuck in "Activating" state forever.

Following error seen in logs:
2018/09/08 00:02:36 [INFO] namespaceHandler: addProjectIDLabelToNamespace: adding label field.cattle.io/projectId=p-zdwnv to namespace=mytest123
2018/09/08 00:02:36 [ERROR] namespaceHandler: Sync: error adding project id label to namespace err=Operation cannot be fulfilled on namespaces "mytest123": the object has been modified; please apply your changes to the latest version and try again
2018/09/08 00:02:37 [INFO] namespaceHandler: addProjectIDLabelToNamespace: adding label field.cattle.io/projectId=p-zdwnv to namespace=mytest123
2018/09/08 00:02:37 [ERROR] error updating ns mytest123 status: Operation cannot be fulfilled on namespaces "mytest123": the object has been modified; please apply your changes to the latest version and try again
2018/09/08 00:03:03 [INFO] Creating [mytest123/mytest123w] service of type [ClusterIP] with ports [[{default TCP 42 {0 42 } 0}]] for workload deployment:mytest123:mytest123w
E0908 00:03:30.248975       6 streamwatcher.go:109] Unable to decode an event from the watch stream: unable to decode watch event: json: cannot unmarshal string into Go struct field Node.status of type v3.NodeStatus

Tested with v2.1.0-rc1
Not able to reproduce the issue where namespace gets stuck in "Activating" state forever.

Tested with v2.1.0-rc1
Able to reproduce the issue where namespace gets stuck in "Activating" state forever when running automated runs relating to custom cluster provisioning.
{
"actions": {
"move": "…/v3/cluster/c-kvczg/namespaces/test-75261?action=move"
},
"annotations": {
"cattle.io/status": "{\"Conditions\":[{\"Type\":\"InitialRolesPopulated\",\"Status\":\"Unknown\",\"Message\":\"Populating initial roles\",\"LastUpdateTime\":\"\"},{\"Type\":\"ResourceQuotaInit\",\"Status\":\"True\",\"Message\":\"\",\"LastUpdateTime\":\"2018-09-18T05:07:08Z\"}]}",
"lifecycle.cattle.io/create.namespace-auth": "true"
},
"baseType": "namespace",
"created": "2018-09-18T05:07:07Z",
"createdTS":<PHONE_NUMBER>000,
"creatorId": "user-2v8lz",
"description": "test",
"id": "test-75261",
"labels": {
"cattle.io/creator": "norman",
"field.cattle.io/projectId": "p-lmgz5"
},
"links": {
"remove": "…/v3/cluster/c-kvczg/namespaces/test-75261",
"self": "…/v3/cluster/c-kvczg/namespaces/test-75261",
"update": "…/v3/cluster/c-kvczg/namespaces/test-75261",
"yaml": "…/v3/cluster/c-kvczg/namespaces/test-75261/yaml"
},
"name": "test-75261",
"projectId": "c-kvczg:p-lmgz5",
"state": "activating",
"transitioning": "yes",
"transitioningMessage": "Populating initial roles",
"type": "namespace",
"uuid": "b0f90a72-bb00-11e8-9760-029026ef3182"
}

Tested with latest build from master
Did not see the issue of namespace getting stuck in "Activating" state forever.

@sangeethah
I am facing this issue with Rancher 2.1.0. I have deployed 3 custom hosts and spin a cluster up. I could install some workloads in system project, but then creating namespaces are stuck in activating state doesn't matter in which project they are created. API view looks like this for one of the stuck namespaces:
{
"actions": {
"move": "…/v3/cluster/c-5rfj5/namespaces/nextcloud?action=move"
},
"annotations": {
"cattle.io/status": "{\"conditions\": [{\"type\": \"InitialRolesPopulated\", \"status\": \"Unknown\", \"message\": \"Populating initial roles\"},{\"type\": \"ResourceQuotaValidated\", \"status\": \"Unknown\", \"message\": \"Validating resource quota\"}]}"
},
"baseType": "namespace",
"created": "2018-10-08T12:35:41Z",
"createdTS":<PHONE_NUMBER>000,
"creatorId": "user-gq6tr",
"id": "nextcloud",
"labels": {
"cattle.io/creator": "norman"
},
"links": {
"remove": "…/v3/cluster/c-5rfj5/namespaces/nextcloud",
"self": "…/v3/cluster/c-5rfj5/namespaces/nextcloud",
"update": "…/v3/cluster/c-5rfj5/namespaces/nextcloud",
"yaml": "…/v3/cluster/c-5rfj5/namespaces/nextcloud/yaml"
},
"name": "nextcloud",
"projectId": "c-5rfj5:p-5hnhn",
"resourceQuota": { },
"state": "activating",
"transitioning": "yes",
"transitioningMessage": "Populating initial roles",
"type": "namespace",
"uuid": "ab20a490-caf6-11e8-ad59-4ed6ddea6d45"
}

I have the same on an imported custom (kubespray) cluster (on premises)
"actions": {
"move": "…/v3/cluster/local/namespaces/xxxxxxx-supporting-prd?action=move"
},
"annotations": {
"cattle.io/status": "{"conditions": [{"type": "InitialRolesPopulated", "status": "Unknown", "message": "Populating initial roles"}]}"
},
"baseType": "namespace",
"created": "2018-10-09T22:52:17Z",
"createdTS":<PHONE_NUMBER>000,
"creatorId": "u-2226z",
"id": "xxxxxxx-supporting-prd",
"labels": {
"cattle.io/creator": "norman"
},
"links": {
"remove": "…/v3/cluster/local/namespaces/xxxxxxx-supporting-prd",
"self": "…/v3/cluster/local/namespaces/xxxxxxx-supporting-prd",
"update": "…/v3/cluster/local/namespaces/xxxxxxx-supporting-prd",
"yaml": "…/v3/cluster/local/namespaces/xxxxxxx-supporting-prd/yaml"
},
"name": "xxxxxxx-supporting-prd",
"projectId": "local:p-w6xsb",
"state": "activating",
"transitioning": "yes",
"transitioningMessage": "Populating initial roles",
"type": "namespace",
"uuid": "f890c9f5-cc15-11e8-934f-000c2908ce1e"
}

{ "actions": { "move": "…/v3/cluster/local/namespaces/identity?action=move" }, "annotations": { "cattle.io/status": "{\"conditions\": [{\"type\": \"InitialRolesPopulated\", \"status\": \"Unknown\", \"message\": \"Populating initial roles\"}]}" }, "baseType": "namespace", "created": "2018-10-11T12:20:02Z", "createdTS":<PHONE_NUMBER>000, "creatorId": "user-c5xx5", "id": "identity", "labels": { "cattle.io/creator": "norman" }, "links": { "remove": "…/v3/cluster/local/namespaces/identity", "self": "…/v3/cluster/local/namespaces/identity", "update": "…/v3/cluster/local/namespaces/identity", "yaml": "…/v3/cluster/local/namespaces/identity/yaml" }, "name": "identity", "projectId": "local:p-2cjg8", "state": "activating", "transitioning": "yes", "transitioningMessage": "Populating initial roles", "type": "namespace", "uuid": "facd0205-cd4f-11e8-8dba-005056987f2c" }
We are now experiencing this behaviour after upgrading to 2.1.0 from 2.0.8.
RKE provisioned imported cluster.

The same after moving to 2.1.0. Can create namespace with kubectl but trying to move it to any project got 502 gateway error.

same problem for my imported cluster with 3 nodes:
{
"actions": {
"move": "…/v3/cluster/local/namespaces/test?action=move"
},
"annotations": {
"cattle.io/status": "{"conditions": [{"type": "InitialRolesPopulated", "status": "Unknown", "message": "Populating initial roles"},{"type": "ResourceQuotaValidated", "status": "Unknown", "message": "Validating resource quota"}]}"
},
"baseType": "namespace",
"created": "2018-11-06T18:32:41Z",
"createdTS":<PHONE_NUMBER>000,
"creatorId": "user-8dzj2",
"id": "test",
"labels": {
"cattle.io/creator": "norman"
},
"links": {
"remove": "…/v3/cluster/local/namespaces/test",
"self": "…/v3/cluster/local/namespaces/test",
"update": "…/v3/cluster/local/namespaces/test",
"yaml": "…/v3/cluster/local/namespaces/test/yaml"
},
"name": "test",
"projectId": "local:p-7q9s2",
"resourceQuota": { },
"state": "activating",
"transitioning": "yes",
"transitioningMessage": "Populating initial roles",
"type": "namespace",
"uuid": "58493007-e1f2-11e8-ac98-c6e96cacb90b"
}

It's documented on docs.rancher.com and in the file you are editing that RKE install is only supported up to v2.0.8, you need the workaround mentioned in https://github.com/rancher/rancher/issues/16251 to make it work.
        # Rancher install via RKE addons is only supported up to v2.0.8
        - image: rancher/rancher:v2.1.1

This is a closed issue, if you have a reproducible situation using the official docs, please open a new issue.


Broken Preview Function for Draft Post

Now the render_post checks if @post.can_visit? , but that method checks if object.status == 'published'. This doesn't allow to preview draft post since the post status is "draft" not "published". The change was introduced in https://github.com/owen2345/camaleon-cms/commit/782f73334dd045c66d660747141bdfd71bc1ea2c .
https://github.com/owen2345/camaleon-cms/blob/393288351730172ed4756574097efe9419ff2ab8/app/controllers/camaleon_cms/frontend_controller.rb#L166
https://github.com/owen2345/camaleon-cms/blob/393288351730172ed4756574097efe9419ff2ab8/app/decorators/camaleon_cms/post_decorator.rb#L193

have you found solution for this one?

im currently tracing the camaleon source code trying to fix how this draft post can viewable

@choyno You can look into the draft_render method in FrontendController. I made a change in that method like:
    if args[:permitted] || can?(:update, post_draft)
      render_post(post_draft, false, nil, true) # add the last argument as flag of draft_render

and also override the render_post method to accept the draft_render flag arg:
def render_post(post_or_slug_or_id, from_url = false, status = nil, draft_render = false)
  ...
   if<EMAIL_ADDRESS>|| (!draft_render && !@post.can_visit?)
      if params[:format] == 'html' || !params[:format].present?
        page_not_found
  ...
end

This is not very elegant but allows draft preview.

@ericbeansmile  what rails version and rails version  you are using?

@choyno The visibility filter is used in many places. I'm not sure there would be any side effect if it allows draft post. Haven't dig too far.


Upgrade Angular to version 7

Upgrades Angular to version 7.2.3 and fixes associated issues.
This is to make a first time install easier because previous versions were out of sync.

Ooooo great idea! This should also increase performance from what you mentioned yesterday. I'll test it out today.

Looks good to me! Didn't break anything it appears. Nice work!


clone 安装 vant-demo/vant-Weapp出错，资源已转移？

如题：
PS D:\Dev\vant\vant-demo\vant-weapp\base> npm -v
9.8.1
PS D:\Dev\vant\vant-demo\vant-weapp\base> npm i
npm WARN old lockfile
npm WARN old lockfile The package-lock.json file was created with an old version of npm,
npm WARN old lockfile so supplemental metadata must be fetched from the registry.
npm WARN old lockfile
npm WARN old lockfile This is a one-time fix-up, please be patient...
npm WARN old lockfile
npm ERR! code ETIMEDOUT
npm ERR! syscall connect
npm ERR! errno ETIMEDOUT
npm ERR! network request to http://registry.npm.qima-inc.com/<EMAIL_ADDRESS>failed, reason: connect ETIMEDOUT <IP_ADDRESS>:80, reason: connect ETIMEDOUT <IP_ADDRESS>:80
npm ERR! network This is a problem related to network connectivity.
npm ERR! network In most cases you are behind a proxy or have bad network settings.
npm ERR! network
npm ERR! network If you are behind a proxy, please make sure that the
npm ERR! network 'proxy' config is set properly.  See: 'npm help config'
详见下图。

把 registry.npm.qima-inc.com 换成 registry.npmjs.org 试试

感谢提醒，还以为是有赞的私链。
官方链接不行，但国内 mirror 通过。


minitest migration part 4

Adds:
test/closure_tree/label_test.rb
Removes:
spec/closure_tree/label_spec.rb

pinging @seuros


Issues in going through the provided data

When i went through the provided data with the code, i got stuck at accessing classifier.
After entering the code " tm_heldoutassessment = assess_comm(ct_scores = classRes_val_all, stTrain = stTrain, stQuery = stTest, dLevelSID = "cell", classTrain = "newAnn", classQuery = "newAnn", nRand = 50)", the following error message popped up:
Error in AUC(tmp$recall, tmp$precision, method = AUCmethod) :
unused argument (method = AUCmethod)

Hi, it should just be a loading library(DescTools) problem. I also altered the code in Github to enforce that function to work. Let me know if you still have errors.

You can install from github again (without the ref tag) or library(DesTools). either option should work.


Output CSV

output the LAST row of the header and bodies of tables as CSV

Implemented, superficially tested and committed.


NTR: a variety of food terms

Hello,
I've created a ROBOT template for my proposed terms here and am very open to any feedback you have. These are requested terms that stem from alignment efforts between an NIEHS survey about food and exposures and FoodOn terms. I know these foods are also not all aligned with one another, so if they should be organized in a different fashion please let me know. Thanks!

Seems like these are all sorted!


[MINOR] Fix case of externalFunction classname message

Use constant for 'classname' in externalFunction error message, since
the previous 'className' in the error message does not work.

Can test with:
time = externalFunction() return (double B)
  implemented in (classname="org.apache.sysml.udf.lib.TimeWrapper");
t = time();
print("Time: " + t);

(Using className instead of classname does not work.)

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/1991/

Thanks for reviewing this @gweidner


Create Samples

Samples - Have 4 different samples :

Simple web example
Web example with data driven
Simple native example
Simple native example with data driven Native examples should work both on android and ios.

Each sample name should be self explanatory, so user will understand what does the sample shows by its name

We need to have a sample of one script executing on both (Desktop web & mobile) as an example too.
Also in the link below the Pre-Defined steps are not mentioned for Web separately.
https://github.com/Project-Quantum/Quantum-Starter-Kit/wiki/Perfecto predefined steps
I understand that a few of the “application” steps can be used for Web too, but it does not come out clearly / explicitly. Maybe just one more column (can be used for Desktop Web – with yes/no) would also suffice.


Assignment1 -Secret Password

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Updated commands used to find the Secret password and the final answers
What did you learn from the changes you have made?
Learned how the shell commands can be used to navigate around the file system and also used to create/edit/delete/modify multiple files in very short amount of time.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
I was trying to find some command to search a string and delete files in the same command but this may need some conditional commands which I intend to learn in the future.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Knowing how powerful shell commands can beI was worried I would delete something. I was cautious when I used the * command as warned.
How were these changes tested?
I tested these commands in the Visual studio terminal. The CAT and ls commands also helped to test the changes.
A reference to a related issue in your repository (if applicable)
@mentions of the person or team responsible for reviewing proposed changes (At least 2 people)

@dtxe
@Emma Teng
@Julia Gallucci
@michaela drouillard
Checklist

[x] I can confirm that my changes are working as intended

@dtxe
https://github.com/emma Teng
@julia Gallucci
@michaelA drouillard


Recaptcha host and request host do not match. Forgery attempt? v2-invis or v3

Hi,
I'm using v2.1.1 due to my ASP.Net Core is version 2.1, giving me troubles with upper versions. I need to use invisible captcha v2-invisible or V3, I tried to follow documentation, follow is Startup:
var recaptcha = Configuration.GetSection("RecaptchaSettings");
            if (!recaptcha.Exists())
                throw new ArgumentException("Missing RecaptchaSettings in configuration.");

services.Configure<RecaptchaSettings>(Configuration.GetSection("RecaptchaSettings"));
services.AddTransient<IRecaptchaService, RecaptchaService>();

Following is login view:
<form asp-area="Identity" asp-controller="Account" asp-action="Login" method="post" class="ffl-form" id="loginForm">
@Localizer["Ingrese al portal de SicotX"]

@Localizer["con sus Credenciales"]

                                     @Localizer["Usuario"]                                     <input asp-for="Usuario" class="form-control" autocomplete="off" />

                                     @Localizer["Contraseña"]                                     <input asp-for="Contraseña" class="form-control" />

                                 @Html.Recaptcha(RecaptchaSettings?.Value)

                                 <button id="recaptcha" type="submit" class="btn btn-primary">@Localizer["Ingresar"] </button>
                         </form>
Following is Account controller for Login action:
public async Task<IActionResult> Login(LoginViewModel model)         {             var recaptcha = await _recaptcha.Validate(Request);             if (!recaptcha.success)             {                 ModelState.AddModelError("Error", _localizer["Hubo un error de validación del Captcha. Por favor intente nuevamente!"].Value);                 _logger.LogInformation($"Hubo un error de validación del Captcha.");             }
I configured appSettings (I created 2 sites for v2 invisible and v3) with the correct sitekey and secretkey and version: v2-invis and throws the exception, as follows:
ValidationException: Recaptcha host, and request host do not match. Forgery attempt?
reCAPTCHA.AspNetCore.RecaptchaService.Validate(HttpRequest request, bool antiForgery) in RecaptchaService.cs
SicotX.Areas.Identity.Controllers.AccountController.Login(LoginViewModel model) in AccountController.cs
+
var recaptcha = await _recaptcha.Validate(Request);
Microsoft.AspNetCore.Mvc.Internal.ActionMethodExecutor+TaskOfIActionResultExecutor.Execute(IActionResultTypeMapper mapper, ObjectMethodExecutor executor, object controller, object[] arguments)
System.Threading.Tasks.ValueTask.get_Result()
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeActionMethodAsync()
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeNextActionFilterAsync()
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ActionExecutedContext context)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(ref State next, ref Scope scope, ref object state, ref bool isCompleted)
Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeInnerFilterAsync()
Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeNextResourceFilter()
Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Rethrow(ResourceExecutedContext context)
Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.Next(ref State next, ref Scope scope, ref object state, ref bool isCompleted)
Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeFilterPipelineAsync()
Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeAsync()
Microsoft.AspNetCore.Builder.RouterMiddleware.Invoke(HttpContext httpContext)
Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware.Invoke(HttpContext context)
Microsoft.AspNetCore.Localization.RequestLocalizationMiddleware.Invoke(HttpContext context)
SicotX.Startup+<>c+<b__6_1>d.MoveNext() in Startup.cs
+
await next();
SicotX.Startup+<>c+<b__6_0>d.MoveNext() in Startup.cs
+
await next();
Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.MigrationsEndPointMiddleware.Invoke(HttpContext context)
Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.DatabaseErrorPageMiddleware.Invoke(HttpContext httpContext)
Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.DatabaseErrorPageMiddleware.Invoke(HttpContext httpContext)
Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)
I changed to correct sitekey and secretkey and v3 but for some reason my submit button dissapeared, no way to click on Login button, and javascript error is thrown:
Uncaught Error: reCAPTCHA placeholder element must be an element or id
at Object.Fd [as render] (www.gstatic.com/recaptcha/api2/v1554100419869/recaptcha__es_419.js:526)
at _eebca1b4_7f0d_4e74_b2e9_d5fba0580a53 (Login:268)
at $p (www.gstatic.com/recaptcha/api2/v1554100419869/recaptcha__es_419.js:516)
at iG (www.gstatic.com/recaptcha/api2/v1554100419869/recaptcha__es_419.js:517)
at www.gstatic.com/recaptcha/api2/v1554100419869/recaptcha__es_419.js:530
at www.gstatic.com/recaptcha/api2/v1554100419869/recaptcha__es_419.js:549
I don't know if I'm following nuget package documentation it not works. What am I doing wrong?
I would prefer to use V3, but not succeded. Thanks for any help!

Hi, there a few reasons this can occur. The first was i think a bug might have existed in this version that was including port number with host name in non-standard situations (such as running in IIS Express). The other is that your using some sort of proxy, or hosting situation in which the domain that google returns from the recaptcha key, and the one your actually hosting on don't match.
In the event this is because of a bug, or if it's a forgery issue you can't resolve. You can simply turn this feature off by setting antiForgery to false with the Reacaptcha helper method. This will prevent the check from occurring at all.
Finally, the latest version of this library no longer targets .NET Core specifically. Rather, it now targets .NET Standard 2.0 which should work in any version of .NET Core 2.x, and most versions of .NET framework (I think 4.6+?)

Sorry to correct one error above. The actual antiForgery boolean is on the Validation method in your controller and not actually in the view options. My apologies for the mistake.

Thanks, I could solved, I migrated my Asp.Net core 2.1 solution to 2.2 which let me update reCaptcha nuget package to the last version available and with those things reCaptcha worked fine! It seems migrating to 2.2 its better than try to find a workaround about it with older nuget package.

How is the antiForgery meant to be set?
In my markdown I have @(Html.Recaptcha<RecaptchaV2Checkbox>(RecaptchaSettings.Value, new RecaptchaV2Checkbox { SuccessCallback = "enableSubmit", ExpiredCallback = "disableSubmit" })) and on my controller I use the annotation [ValidateRecaptcha(score: 0.7)] and neither of these take in a antiForgery argument.


chore(deps): update lb4 dependencies

updated lb4 dependencies
gh-22

:tada: This PR is included in version 3.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


BUG: Improve the footer section.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

I think the marked parts in the footer section needs to be improved. Please assign this task to me under the gssoc and level labels.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@arghadipmanna101 please assign this task to me. I'm willing to do the changes. Also, please add the gssoc and level label to it.


Network Error when trying to use PKCE with Angular SPA and Azure Active Directory

Hi!
I am trying to use this package for getting PKCE to work with an Angular SPA with Azure Active Directory.
I am using the following blog posts for reference -

Initial Setup - https://www.scottbrady91.com/Angular/SPA-Authentiction-using-OpenID-Connect-Angular-CLI-and-oidc-client
PKCE Setup  (Just changing the response_type to code) - https://www.scottbrady91.com/Angular/Migrating-oidc-client-js-to-use-the-OpenID-Connect-Authorization-Code-Flow-and-PKCE

The problem is that when I am using the Access Tokens or ID Tokens workflow, the setup is working fine and I am able to hit the backend API. But, when I change the response_type to code, I am getting a Network Error like below -

I checked the network tab and I seem to be successfully getting the Azure Active Directory Bearer Token using the PKCE flow but because of the above, I am not able to query the backend API. It spits out the error while doing the following -
completeAuthentication(): Promise<void> {
      return this.manager.signinRedirectCallback().then(user => {
          this.user = user;
      });
  }

Any idea what I might be doing wrong?
Thanks,
Pranav

Can you provide more info on the request/response payload for the token endpoint HTTP call?

@brockallen - Please find the information below (I have edited some of the tokens and GUIDs to remove potentially sensitive information -
POST
https://login.microsoftonline.com/b1519f0f-aaaa-aaaa-aaaa-a686ce97588a/oauth2/v2.0/token

Request Headers -
Host: login.microsoftonline.com
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:71.0) Gecko/20100101 Firefox/71.0
Accept: */*
Accept-Language: en-US,en;q=0.5
Accept-Encoding: gzip, deflate, br
Content-Type: application/x-www-form-urlencoded
Content-Length: 993
Origin: https://localhost:4200
DNT: 1
Connection: keep-alive
Referer: https://localhost:4200/

Response Headers -
HTTP/1.1 200 OK
Cache-Control: no-cache, no-store
Pragma: no-cache
Content-Type: application/json; charset=utf-8
Expires: -1
Strict-Transport-Security: max-age=31536000; includeSubDomains
X-Content-Type-Options: nosniff
x-ms-request-id: 6fbba727-1faa-4710-ab65-d5ce90728100
x-ms-ests-server: 2.1.9767.22 - SAN ProdSlices
Referrer-Policy: strict-origin-when-cross-origin
P3P: CP="DSP CUR OTPi IND OTRi ONL FIN"
Set-Cookie: fpc=ArYcKccR2xhAmH0bPVDWXoU8kAppAQAAAFNiktUOAAAA; expires=Wed, 22-Jan-2020 07:23:00 GMT; path=/; secure; HttpOnly; SameSite=None
Set-Cookie: x-ms-gateway-slice=prod; path=/; SameSite=None; secure; HttpOnly
Set-Cookie: stsservicecookie=ests; path=/; SameSite=None; secure; HttpOnly
Date: Mon, 23 Dec 2019 07:23:00 GMT
Content-Length: 2589

Request Payload
client_id=baf7d4b2-aaaa-aaaa-aaaa-10464c3b8c67&code=OAQABAAIAAACQN9QBRU3jT6bcBQLZNUj7knCH6fdDWS6mHIRC7Azp7jaPuJhNc3itJUHsXTV3edVb_qUVndtP-H2xUq-VP9cJWX5nGOQgc3AVk4yncUHFTMOeIS1na1i1rWs_IWI69Oaw4ddbbGVPWxi0ZeJRYIb7CaMO8ews4eIEbFjZ_Eo3M52kKq8KlTYu5ZjqbFBl7XMKxV-II33NvGtVTiY-wT12pm1xEfZncGoPgwvmijC2FFumlbOXFNym231B879CPaM19OK8HCx8OYYWZtFKkyyQDGN3YfOoWtiLp25DEqe9h5TWmiLcN6kIwx9svjYwFiLCCTWiM6hEaAdr1njVR9t-YrkoCnGJdka8nfkyneyds0D5sdwrCR9JnYcSfcgYUsjdpzxsEEZ_vt-CkmRHa_nJtN-Lwo2bG0JX3GxWx8OgT8xK0tyfWiiJIhCOy-YAC_16qnY0TQCYx31tz9R35hLirCFNC-QNS8bjvUIr-UHLIel8H04CgYTlL3GVyyqHRrKMTVNlQmY6JC_Yn9qcxfsrMTF8mlbCbzRpZ4t87oxB1lNldke640pgERxdPcuG1OKXtvmsNNFltI6kghDrKydZeHXX0rvmrJDisL5AhsIv8l8NRvjxNevme3lBojFIUGB6B6cVLFZiRk1iQWeR4Xr21ojwrJZF3sslP6zFiRxU0Xa3ujTjvPkvgmliMm9Q3EKpccO8WWBzU-yxdFVKTC_AIAA&redirect_uri=https%3A%2F%2Flocalhost%3A4200%2Fauth-callback&code_verifier=3fc1ca6240484b57b9d72ac9666fdd5ca3fbead3f4df428dbd47c041b8bd0ed8542e348313804feb90d7218111dc4116&grant_type=authorization_code

Response Payload -
{"token_type":"Bearer","scope":"ab10f4de-aaaa-aaaa-aaaa-d6b3ef854364/access_as_user","expires_in":3599,"ext_expires_in":3599,"access_token":"eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsIng1dCI6InBpVmxsb1FEU01LeGgxbTJ5Z3FHU1ZkZ0ZwQSIsImtpZCI6InBpVmxsb1FEU01LeGgxbTJ5Z3FHU1ZkZ0ZwQSJ9.eyJhdWQiOiJhYjEwZjRkZS01NDRlLTQ5YzctYTViOC1kNmIzZWY4NTQzNjQiLCJpc3MiOiJodHRwczovL3N0cy53aW5kb3dzLm5ldC9iMTUxOWYwZi0yZGJmLTRlMjEtYmYzNC1hNjg2Y2U5NzU4OGEvIiwiaWF0IjoxNTc3MDg1NDgwLCJuYmYiOjsImV4cCI6MTU3NzA4OTM4MCwiYWNyIjoiMSIsImFpbyI6IjQyVmdZQ2c0d0xmWFdTRzMzdXQxQXgvYjRSdHFwWFhHTStSL1c1NllyNXNqMjliakoxVVVwZjVjWCt5S3d5c3BZMXVWUitXM0FRPT0iLCJhbXIiOlsicHdkIl0sImFwcGlkIjoiYmFmN2Q0YjItNjE5OS00NGNkLTgyZGItMTA0NjRjM2I4YzY3IiwiYXBwaWRhY3IiOiIwIiwiZ2l2ZW5fbmFtZSI6InByYWNkZXYiLCJpcGFkZHIiOiIxMjguMTk5LjIwMy45NyIsIm5hbWUiOiJwcmFjZGV2Iiwib2lkIjoiMzNlZWI0OTMtOWU0Yy00MzRkLWJiNjktNjZiMjkzZjY2ZDFlIiwib25wcmVtX3NpZCI6IlMtMS01LTIxLTIwOTY3MTE1OTAtMTQzOTUwNTcwMi03NDM3OTQzMDAtNzM4NzYwIiwic2NwIjoiYWNjZXNzX2FzX3VzZXIiLCJzdWIiOiItRWt1Vm9pNVRYcDJqUWJmR3NyU2ttYzgtSlI4NVgxUGlqNzRTRVF6OTNBIiwidGlkIjoiYjE1MTlmMGYtMmRiZi00ZTIxLWJmMzQtYTY4NmNlOTc1ODhhIiwidW5pcXVlX25hbWUiOiJwcmFjZGV2QGF2ZW5kcmEuY29tIiwidXBuIjoicHJhY2RldkBhdmVuZHJhLmNvbSIsInV0aSI6Iko2ZTdiNm9mRUVlclpkWE9rSEtCQUEiLCJ2ZXIiOiIxLjAifQ.gq8M3UFSiuRirz8CnbthtU-y2YCR0g94Cg9crUyjWnJRL6Ky6WgWmBC4iguoyJmpN4RJS816a1539NRfUdcE1Br7GnYgh-EqnOX2rKw9WxL5DgjEXy4ivR50LbjrHrND8NKfATt19IZh7e7ZP4fhuxlJXxBHmjsQ0qNi3CMkPMxQNXr5xC4l-sylUyaAV4Qpm4aEtOy8sfCTbO_tg25hgC8Vsd74Fx7gjPaZYbEk-am1sZ_J55MYj1dcDM8dD9vOLVwYQLUmqmiXqzN3aaHr0NmsRMGxLUDT-YJwQ3j8zW8Tta2vIZx0i_8ygieYSEusTSdyrxEPJ5-ZbE6BThzO_Q","id_token":"eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiIsImtpZCI6InBpVmxsb1FEU01LeGgxbTJ5Z3FHU1ZkZ0ZwQSJ9.eyJhdWQiOiJiYWY3ZDRiMi02MTk5LTQ0Y2QtODJkYi0xMDQ2NGMzYjhjNjciLCJpc3MiOiJodHRwczovL2xvZ2luLm1pY3Jvc29mdG9ubGluZS5jb20vYjE1MTlmMGYtMmRiZi00ZTIxLWJmMzQtYTY4NmNlOTc1ODhhL3YyLjAiLCJpYXQiOjE1NzcwODU0ODAsIm5iZiI6MTU3NzA4NTQ4MCwiZXhwIjoxNTc3MDg5MzgwLCJhaW8iOiJBVFFBeS84TkFBQUFFV2ZNc0thaTFnWWZFmS3UzdmtTSmpHQUlVTUhSb2VZbVBoR0RLc0xJK0ZSbUVEdWgzdVZNdktLIiwic3ViIjoiUDNoOEs2N19iTWxJbE9pdGdHZEJPTmEyeVNYM3pWSHBINlNqS0IwbVd0ayIsInRpZCI6ImIxNTE5ZjBmLTJkYmYtNGUyMS1iZjM0LWE2ODZjZTk3NTg4YSIsInV0aSI6Iko2ZTdiNm9mRUVlclpkWE9rSEtCQUEiLCJ2ZXIiOiIyLjAifQ.AD3SAUl1Rv0U8b9IsgYk5dI78fuPcaB0FH1WfTKXNQLx30yoSH2YOgAgK7FoqiqfWVvPhc2laNvYjBSwjJuzc77VopdQ7ogRA_h2seYW7IDWOq_8w4C9ThYxWnpbx6zqa6WX_31MIFUj1AzzhVqSqPc7ltJH7bvzUEXFZiOru5bUfQQ4ot62qE6fzfvhOMbIWoVFB0gn1odX44-n2RYaNJ063gmGxV5SeN8Ru8o3qI4XlyPmyIx_k86FEoPFQ5nZQjxZJkgQyEmQtVlPJlQimCmeXxAnatrfQCFHR4RAdFeTfA_0WbBmJLf2OdGahg2Mb7IewUKFzje1anYMJkP6Yw"}

Here is the config which is being used in the code -
export function getUserManagerSettings(): UserManagerSettings {
  return {
        authority: 'https://login.microsoftonline.com/b1519f0f-aaaa-aaaa-aaaa-a686ce97588a',
        client_id: 'baf7d4b2-aaaa-aaaa-aaaa-10464c3b8c67',
        redirect_uri: 'https://localhost:4200/auth-callback',
        post_logout_redirect_uri: 'https://localhost:4200/',
        response_type: 'code',
        response_mode: 'query',
        scope: 'openid ab10f4de-aaaa-aaaa-aaaa-d6b3ef854364/access_as_user',
        filterProtocolClaims: false,
        loadUserInfo: false,
        metadataUrl: 'https://login.microsoftonline.com/b1519f0f-aaaa-aaaa-aaaa-a686ce97588a/v2.0/.well-known/openid-configuration'
  };
}

In Azure AD, is your app registered as a native / desktop app? Checking since last time I checked, client_secret is required for web apps with the code flow.
In the portal, you can check this under app registration > authentication > Redirect URIs. The redirect url's type should be Public client / Native for the PKCE to work on Azure AD.
The explanation is found here, on the item client_secret.

@jorgecarleitao - Yep. The redirect URI is set to "Public Client/Native". As mentioned above, the PKCE flow seems to work because I am getting the code and token. Just seems as if there is some issue in the code with this library in saving it properly?

I was able to reproduce this issue on the vanilla example. I pushed the changes to reproduce the issue here.
The steps to reproduce are:

npm start
go to http://localhost:3000/user-manager-sample.html
press start signin main window
authenticate
press end signin main window

This will cause the Network error that @blueelvis is describing.

@jorgecarleitao thx for that. i'll have a look when i can

I have some time, so I spent some time investigating this.
My conclusion is that this is an error on the AD side.
Some facts:

The error is triggered on this line because the request's status is !== 200
The request is completed (readyState === 4) and its status is 0
status 0 is not a valid HTTP status, so it can never be sent by the server.
As per this SO question, when the header for CORS is not present XMLHttpRequest can have a status 0.
The browser shows an error about a missing CORS header on the request to /token.
The browser shows that the response is status 200.
The response does not contain the header Access-Control-Allow-Origin

Overall, this explains why the browser receives the response with status 200 and the token, but it does not pass it down to the js: the server is telling it not to.
So, it seems to me that Microsoft is not allowing CORS requests to the /token endpoint, which is counter the whole idea of PKCE on clients, of as far as I can understand of OAuth2.
Could someone with more experience of this chip in and validate/refute this hypothesis? If valid, we should raise it on Microsoft side and close this issue.

Could someone with more experience of this chip in and validate/refute this hypothesis? If valid, we should raise it on Microsoft side and close this issue.

This all sounds legit to me. I know they didn't support CORS on the discovery endpoint for the longest time. It'd not surprise me if they didn't support CORS on the token endpoint.

I asked someone who might know: https://twitter.com/BrockLAllen/status/1209178114804375552

EDIT 2: upon further investigation, PKCE for SPA is not supported by Microsoft AD, as per this discussion and in particular this comment. I thus recommend that we close this as won't fix.

Ah, sorry -- I missed this last comment. Thanks for looking into it.

@brockallen @jorgecarleitao - Thanks a lot guys for the detailed explanation and investigation on this! Just curious but is it possible to catch the situation particularly about the CORS and mention in the logs?  Otherwise, the way the log is written, it seems quite something else.

Well, from the http error you can't quite tell that's the exact reason.


Crazyflie Optical Flow EKF2 bringup

➤ Thomas Gubler commented:
@Daniele Pettenuzzo can youdescrive your setup and issues here please

➤ Daniele Pettenuzzo commented:
optical flow driver: pmw3901 (external SPI bus CS1)
distance sensor driver: vl53lxx (external I2C bus)

➤ Daniele Pettenuzzo commented:
drivers for optical flow board are in pr-bitcraze_flow_rebase

➤ Daniele Pettenuzzo commented:
these changes are:

added read_first_older_than function in RingBuffer.h (to read gyro data without moving head of buffer)
modified setOpticalFlowData in estimator_interface: we need to fetch gyro data because optical flow board doesn't have its own gyro (this obviously breaks the px4flow but it is just for testing)
minor changes in the optflow_fusion

@DanielePettenuzzo Can you comment (on github ;) ) about the issues you see with EKF2 and the above setup so that we can have Paul look into it?


Intercept and format user messages

Is it possible to intercept user messages and prepend/append text to them? Ideally I'd like to take every message coming in and, depending on the user, prepend text to their message (like a user ID) before allowing the message to reach the channel/private chat.
(Sorry if this is the wrong place to ask questions like this!)

@SpencerKaiser if I'm understanding you right, you want hubot to receive messages from users, and depending on the user, change the message before it's delivered to the room?
If that's the case, I don't think it's possible, as the message is reaching both the channel/private chat at the same time that hubot is receiving it.
Or, do you mean, when sending messages back to users, you'd like to append something to the hubot message the user will receive? That would be possible with hubot response middleware

Hey @technicalpickles Thanks for the response! Since I posted this question I've given up, but yea it was the former and I figured it was not possible.


[IOTDB-3550]Fix show child nodes

Description
See IOTDB-3550
The duplicate result has been fixed in #6530.
Besides, there is still NPE problem, which is fixed in this pr.

Hi, I met this issue as well, I think 'show child paths' still have problem. The root cause for NPE is in this PR #6592

OK, I will have a check.


Fix #359 Safari's Can't find variable: Intl error on startup

Fixes Issue #359

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

Thanks!


Failed to create sensor from bridge model: Error: Unknown type 'zllrelativerotary' to create Bridge Model Object

Failed to create sensor from bridge model: Error: Unknown type 'zllrelativerotary' to create Bridge Model Object from.; payload:

 {     "state": {         "rotaryevent": 2,             "expectedrotation": 15,                 "expectedeventduration": 400,                     "lastupdated": "2022-12-02T18:54:25"     },     "swupdate": {         "state": "noupdates",             "lastinstall": "2022-10-30T18:59:33"     },     "config": {         "on": true,             "battery": 100,                 "reachable": true,                     "pending": [                     ]     },     "name": "Hue tap dial switch 1",         "type": "ZLLRelativeRotary",             "modelid": "RDM002",                 "manufacturername": "Signify Netherlands B.V.",                     "productname": "Hue tap dial switch",                         "diversityid": "e5b68d4b-0000-436d-88c5-7e02ffabda37",                             "swversion": "2.59.25",                                 "uniqueid": "00:17:00:01:0d:00:bc:de-01-fc00-0014",                                     "capabilities": {         "certified": true,             "primary": true,                 "inputs": [                     {                         "repeatintervals": [                             400                         ],                         "events": [                             {                                 "rotaryevent": 1,                                 "eventtype": "start"                             },                             {                                 "rotaryevent": 2,                                 "eventtype": "repeat"                             }                         ]                     }                 ]     } }

Now I suppress error messages by comment line
.\node-hue-api\dist\cjs\api\http\endpoints\sensors.js:104
Thanks for any help.

Great! Thank you very much! 👍
I will check this tomorrow and respond with feedback. :)

💯 Approved! Thanks for your help.
Log of my (Hue tap dial switch) sensor:
[69][Hue tap dial switch 1]             lastupdated: "2022-12-15T11:11:53"      rotaryevent: 1  expectedrotation: 0     expectedeventduration: 400
[69][Hue tap dial switch 1]             lastupdated: "2022-12-15T11:11:53"      rotaryevent: 1  expectedrotation: -15   expectedeventduration: 400
[69][Hue tap dial switch 1]             lastupdated: "2022-12-15T11:11:56"      rotaryevent: 1  expectedrotation: 15    expectedeventduration: 400
[70][Hue tap dial switch 2]             lastupdated: "2022-12-15T11:11:58"      buttonevent: 4002
[70][Hue tap dial switch 2]             lastupdated: "2022-12-15T11:11:59"      buttonevent: 2000
[70][Hue tap dial switch 2]             lastupdated: "2022-12-15T11:11:59"      buttonevent: 2002
[70][Hue tap dial switch 2]             lastupdated: "2022-12-15T11:12:01"      buttonevent: 1002
[70][Hue tap dial switch 2]             lastupdated: "2022-12-15T11:12:02"      buttonevent: 3002
[70][Hue tap dial switch 2]             lastupdated: "2022-12-15T11:12:04"      buttonevent: 4002
[69][Hue tap dial switch 1]             lastupdated: "2022-12-15T11:12:06"      rotaryevent: 1  expectedrotation: -15   expectedeventduration: 400
[71][Hue dimmer switch 1]               lastupdated: "2022-12-15T11:12:08"      buttonevent: 3002
[71][Hue dimmer switch 1]               lastupdated: "2022-12-15T11:12:09"      buttonevent: 2002


Add test for glance

Could please also compare results with output from this script:
https://github.com/openstack/timmy/blob/master/timmy_data/simplified-performance-testing/scripts/glance-1-image-create.sh ? We need to be sure that python glance client doesn't affect measured values. Thank you.


[BUG] no-op with runSQL migrations

Describe the bug
I use migrations.runSQL() to perform some migrations that the ORM cannot support for clickhouse DB. And it not work.
The migrations code in clickhouse_backend/patch/migrations.py does not include special migrations of Django (from django.db.migrations.operations.special)
Is it a bug or you guys dont want to perform that action?
# Generated by Django 3.2.14 on 2024-07-11 12:49

from django.db import migrations

class Migration(migrations.Migration):

    dependencies = [
        ('api', '0001'),
    ]

    operations = [
        migrations.RunSQL(
            'ALTER TABLE "event"  ADD COLUMN "record" String CODEC(ZSTD(6))'
        )
    ]

Versions

ClickHouse server version. Version can be obtained by running SELECT version() query.
Python version.
Clickhouse-driver version.
Django version.
Django clickhouse backend version.

duplicate #15

Do you have any way to decrease the memory, it use a lot of mem, I try to
set connection pool lower but not affect
On Sat, Jul 13, 2024 at 9:25 AM jayvynl @.***> wrote:

duplicate #15
https://github.com/jayvynl/django-clickhouse-backend/issues/15
—
Reply to this email directly, view it on GitHub
https://github.com/jayvynl/django-clickhouse-backend/issues/91#issuecomment-2226725733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALV2XBJYCTKH7VTQHQPE2ULZMCFYHAVCNFSM6AAAAABKYDAUAOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRWG4ZDKNZTGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

one more question, when will you update the clickhouse-driver version to
0.2.8
I found the issue is because the date/date32 field in taking a lot of
memory. cre:
https://clickhouse-driver.readthedocs.io/en/latest/types.html#date-date32
On Tue, Aug 6, 2024 at 8:36 AM Mai Nguyễn @.***> wrote:

Do you have any way to decrease the memory, it use a lot of mem, I try to
set connection pool lower but not affect
On Sat, Jul 13, 2024 at 9:25 AM jayvynl @.***> wrote:

duplicate #15
https://github.com/jayvynl/django-clickhouse-backend/issues/15
—
Reply to this email directly, view it on GitHub
https://github.com/jayvynl/django-clickhouse-backend/issues/91#issuecomment-2226725733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALV2XBJYCTKH7VTQHQPE2ULZMCFYHAVCNFSM6AAAAABKYDAUAOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRWG4ZDKNZTGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

released as v1.3.0


Add tests for engine and processors

Torii engine is in charge of pulling the events and processing each one of them using the corresponding processor.
More tests have to be added to ensure the modifications made to the engine and processors are correctly done.
The engine is here: https://github.com/dojoengine/dojo/blob/22072dd62792fd3625d5c02c1f10254b6b10f9f5/crates/torii/core/src/engine.rs
The processors are here: https://github.com/dojoengine/dojo/blob/22072dd62792fd3625d5c02c1f10254b6b10f9f5/crates/torii/core/src/processors/mod.rs
Two approaches to write those tests:

Mocking: by not using any underlying node, all processors can first be tested with mocked events, constructed by end, to ensure we can extract correctly the information for a well formatted event.
To test the logic around the event fetching and pagination, the starknet provider can also be mocked, or Katana test runner may be use.

This issue can be tackled in two PR if necessary.

@glihm i will love to work on this issue

Hey @mubarak23, thanks for the will to contribute!
Do you have already an idea on how you will tackle that? Do you have any question on the event content for testing?

@glihm i have some questions,

What are the key components of an event
How do events relate to other entities
What criteria are used to validate whether an event is valid or should be processed

What are the key components of an event

An event is composed of two arrays of felts. keys and data. In the case of keys, the first element is always the event selector (which is a hash that identifies the name of the event). data contains all fields being serialized, if they are not annotated with #[key] attribute (which put them into keys).

How do events relate to other entities

To which entities are you referring to?

What criteria are used to validate whether an event is valid or should be processed

This depends on the processor. In Torii, you'll see that each processor implementation should have a validate function:
https://github.com/dojoengine/dojo/blob/22072dd62792fd3625d5c02c1f10254b6b10f9f5/crates/torii/core/src/processors/store_set_record.rs#L26
The goal of this issue is to organize tests around this.
The principal challenge here is that, generally the processor require to fetch data from the DB and store data to the DB. Ideally, we want those tests to be as unit as we can. Which could imply:

Refactoring the code to have smaller and easily testable function that are not using the DB (for example, a function that only parse the entity from data)
Doing a test as we have in sql_test.rs where we have to run a katana instance, initialize a DB and work around that.

I would say 1 is better, but if too challenging, you can write tests for each of the processors using 2. The downside of two is that the migration is considered very slow, so this will significantly impact the CI.

@glihm i will like to tackle this issue using second approach [2] by following these outlines
- Valid event with correct keys and data.
- Invalid event with incorrect keys or data.
- Error scenarios such as database errors or deserialization failures.
but i will love to hear your though process on how we can achieve 1

Hum, sorry @mubarak23 I've not given any followup here.
Will unassign please let us know if you are still willing to work on that.

@glihm Can I work on this?

@PedroRosalba any idea on how you would tackle that? If yes, all yours!

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have 3 years of experience in back-end development for financial systems. I can use my skills to debug, investigate, and test different sources to solve the issue.
How I plan on tackling this issue
I would try to connect both approaches:
1 - Mocking Approach: Provides controlled unit testing for processors and event recovery logic
2 - Katana Test Runner: Enables comprehensive integration tests that simulate a real StarkNet environment
By combining both approaches, the Torii engine and processors can be thoroughly tested, ensuring their correctness and robustness

hi @glihm, i would love to work on this

hi @glihm, i would love to work on this

All yours, let us know if you have any question. 👍

Torii engine is in charge of pulling the events and processing each one of them using the corresponding processor.
More tests have to be added to ensure the modifications made to the engine and processors are correctly done.
The engine is here: https://github.com/dojoengine/dojo/blob/22072dd62792fd3625d5c02c1f10254b6b10f9f5/crates/torii/core/src/engine.rs The processors are here: https://github.com/dojoengine/dojo/blob/22072dd62792fd3625d5c02c1f10254b6b10f9f5/crates/torii/core/src/processors/mod.rs
Two approaches to write those tests:

Mocking: by not using any underlying node, all processors can first be tested with mocked events, constructed by end, to ensure we can extract correctly the information for a well formatted event.
To test the logic around the event fetching and pagination, the starknet provider can also be mocked, or Katana test runner may be use.

This issue can be tackled in two PR if necessary.

@glihm, i would suggest an hybrid approach.
for testing the core logic of individual processors (BlockProcessor, EventProcessor and TransactionProcessor). we would mock events and providers to verify that the processors can correctly handle and process events.
then, for testing the overall flow of the engine, including event fetching, processing, and pagination. we could use a test runner like Katana to simulate the blockchain environment and ensure that the engine interacts correctly with the blockchain and processes events as expected.

If this is still open, I'd like to step in
I was an early contributor to the Starknet community. After being disconnected for a few months, I’m eager to get back on track, and I’d like to work on this issue as a way to re-engage and contribute actively again.
Aside from that, I’m passionate about seeing Web3 gaming thrive, and I believe Dojo is the best project to make that vision a reality.

hey @bigherc18 im currently working on it


[FEATURE] Generate API code from OpenSearch API Spec

Is your feature request related to a problem?
Related to https://github.com/opensearch-project/opensearch-clients/issues/19
What solution would you like?
Re-use the generator that generates API Code that maintains backwards compatibility with the current client APIs and provides a way to add new APIs.
What alternatives have you considered?
Using OpenAPI generators instead of re-using the existing generator but it makes it difficult to maintain backwards compatibility.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

I would like to help with this issue. Is it still relevant?

Hi @tannerjones4075, yes absolutely! The first step would be use to the API specs from https://github.com/opensearch-project/opensearch-api-specification/blob/main/OpenSearch.openapi.json and modify the existing non-functional generator in the go client to create code similar to the existing client. The general idea is to replace all the code (module by module or divided in whatever way works) in this repo for the APIs with code generated from the API specs without breaking the users.

I am currently trying to generate sturcts out of the component schemas and I don´t think I can continue at this point.
There are lots of object properties defined which no longer exist in the API and are referencing to types that can not be handled in opensearch.
component/schemas/_common:ByteSize is of type oneOf pointing to type number or string.
Inside golang it is not possible for a field to be both. We would need to set the field to json.RawMessage which is just a wrapper for any, forcing the user to find out the type of the field on his own.
_common:ByteSize is referenced quite some time but when going though the the OpenAPI spec and our current golang code, I noticed that many fields that reference _common:BysteSize are no longer returned by the API as they got replaced by the same property suffixed with _in_bytes pointing to type number.
As we validate all our golang structs against the API and check for missing fields, I assume that those fields must be very old as they never existed in any Opensearch 1.x o 2.x release.
For example:
https://github.com/opensearch-project/opensearch-api-specification/blob/9c15bbf1522b1604e2e36f16b53163413f5fe833/spec/schemas/_common.yaml#L982-L986
Should I open an issue for that in the api spec repository? @dblock
Similar issue for component/schemas/_common:Percentage. It is also oneOf for number  or string.
This is used for the cluster health endpoint. Which returns the value as number for yaml and json.
Another issue that we might be able to handle is the use of oneOf pointing to a reference or an array of the same reference. Meaning the field be a direct object/type or and array of that object/type.
https://github.com/opensearch-project/opensearch-api-specification/blob/9c15bbf1522b1604e2e36f16b53163413f5fe833/spec/schemas/_common.yaml#L342-L346
In most cases those schemas are referenced in the parameters where we can assume the array type to work.
Some more information:
Nearly all anyOf are used to mark values as nullable which should be removed once the issue: https://github.com/opensearch-project/opensearch-api-specification/issues/366 is handled.

First, I'm really glad to see you look into this @Jakob3xD!

I am currently trying to generate sturcts out of the component schemas and I don´t think I can continue at this point. There are lots of object properties defined which no longer exist in the API and are referencing to types that can not be handled in opensearch.

If you see bugs/issues in the spec you can/should fix them or open issues, absolutely yes, please. We inherited a lot from Elastic.

component/schemas/_common:ByteSize is of type oneOf pointing to type number or string. Inside golang it is not possible for a field to be both. We would need to set the field to json.RawMessage which is just a wrapper for any, forcing the user to find out the type of the field on his own.

Either the spec is incorrect or the spec is correct and we should handle it. Feel free to open an issue if you're unsure, as always.
_common:ByteSize is referenced quite some time but when going though the the OpenAPI spec and our current golang code, I noticed that many fields that reference _common:BysteSize are no longer returned by the API as they got replaced by the same property suffixed with _in_bytes pointing to type number. As we validate all our golang structs against the API and check for missing fields, I assume that those fields must be very old as they never existed in any Opensearch 1.x o 2.x release.

For example:
https://github.com/opensearch-project/opensearch-api-specification/blob/9c15bbf1522b1604e2e36f16b53163413f5fe833/spec/schemas/_common.yaml#L982-L986
Should I open an issue for that in the api spec repository?

You're definitely right. I opened https://github.com/opensearch-project/opensearch-api-specification/issues/423, but in general please don't hesitate to open issues even if you're unsure.

Another issue that we might be able to handle is the use of oneOf pointing to a reference or an array of the same reference. Meaning the field be a direct object/type or and array of that object/type. https://github.com/opensearch-project/opensearch-api-specification/blob/9c15bbf1522b1604e2e36f16b53163413f5fe833/spec/schemas/_common.yaml#L342-L346 In most cases those schemas are referenced in the parameters where we can assume the array type to work.

One way you can handle this is to roll out custom types without having to parse it or always narrow it down to a primitive type. If something is of FooBarType you could make an actual GoLang struct called FooBar, no?

Some more information: Nearly all anyOf are used to mark values as nullable which should be removed once the issue: opensearch-project/opensearch-api-specification#366 is handled.

Right. LMK which specific issues are blocking you and I will look into them.

To give an update, to better understand the issue and my last comment.
ByteSize
I remove the ByteSize type in the openapi spec and replaced it with a string type component and an integer type component to remote the anyOf reference.
https://github.com/opensearch-project/opensearch-api-specification/pull/552
There is still an open discussion how we can improve the naming.
"Old" fields

There are lots of object properties defined which no longer exist in the API

Looking deeper I notices this statement is false. The fields still exists but are "hidden" behind the human parameter.
I also tried to adjust the referenced in the MR named above.
Percentag
I haven't looked into the Percentage oneOf problem. Maybe this can also be split up into two components with a fixed type.
Null
The nullable field got removed in favor of the null type. https://github.com/opensearch-project/opensearch-api-specification/pull/436
If a component has type null in it, it needs to be a pointer.
Issues (i currently know of)
anyOf/oneOf
Components using anyOf or oneOf need to be handled. For components that use them to not define two components, like it was with the BytesSize component, they should be split into components with a single type.
Components using anyOf or oneOf to reference the component directly or as an array need to be handled.
If the Component is only used as Parameter we could just assume the array type. A "simpler" more fast forward solution wold be to use type any, in combination with an open issue to improve this.
omitempty
How do we know where to set the omitempty flag.
Haven't tough about this much but maybe the required openapi field can help with this.
This is very important  when it comes to request structs. For example you want to update a single filed, then other fields most likely needs to be omitted so they don't overwrite things with empty strings/arrays.

I am currently not working on this, as IMO the openapi spec needs to be improved first to work better with golang.
However you can also prove me wrong that the current openapi spec works fine with golang or you have good ideas to handle the issue I ran into :D


Initial version of reporting dashboard

https://www.pivotaltracker.com/story/show/147783817

created initial version of reporting dashboard

accessible to 'admin' and 'analyst' roles
api endpoint + html template
*includes User Stats and Institution Stats section (no graphs yet, just counts)

for full definitions of sections, see PT story

created 'analyst' role, which only has access to reporting (no PHI)
added new page to dropdown menu (if user has appropriate roles)

Hello @OptimusRhine! Thanks for submitting the PR.

In the file portal/views/portal.py, following are the PEP8 issues :

Line 925:58: E712 comparison to True should be 'if cond is True:' or 'if cond:'
Line 926:29: E127 continuation line over-indented for visual indent
Line 933:54: E712 comparison to False should be 'if cond is False:' or 'if not cond:'

PLEASE DO NOT MERGE YET.
Waiting on:

@mcjustin to review and weigh in on the patient-type definitions
@achen2401 to review (and potentially improve) the template

OK, looks like the definitions for the sub-categories of patients are good for now. This should be ready to merge.


Correct behaviour of timing_response parameter in plugin multi-stim-multi-response

According to documentation timing_reponse = 0 should not result in indefinitely waiting for a response, but end the trial right away.

Thanks for catching this.

As you pointed out in the other thread, there are other plugins with timing_response as a parameter, which were affected by the same bug. I added commits for the three other plugins I could find using this.

Now that I think about this a bit more, I think that the behavior is mostly correct for this portion of the code:
if (trial.timing_response > 0) {
  var t2 = setTimeout(function() {
    end_trial();
  }, trial.timing_response);
}

We should not set a timeout when the timeout value is 0, we should just execute the function immediately. If you call setTimeout with a timeout of 0, there will be some small delay (~20ms?) before the function executes, which is unnecessary in this case.

I see your point, although this behaviour would be consistent with other small numbers for timing_response. If you were to call end_trial() immediately, then there would be a ~20ms difference between setting timing_response to 0ms and 1ms. This could be confusing!
However the confusing thing in this case would probably be the 1ms setting resulting in a ~20ms timeout, not the 0ms = 0ms.
Any chance to get rid of that delay?

After further thought and after running my current experiment and examining the timing of stimulus presentation, I come to the same conclusion. For timing_response = 0, end_trial() should be called immediately. Maybe it should be noted in the documentation, that values >0 and <20 for timing_response are unlikely to be achieved accurately, depending on the browser and system ressources. There are a couple of articles / stackoverflow posts regarding this to be found on google as well.


Make it work

This entails:

[ ] Getting actual emception build artifacts

Apparently the GitHub Pages deployment artifact (the .zip index of everything that got deployed to github pages) expired https://github.com/jprendes/emception/actions/runs/4198818678 but maybe we can get at it some other way? Somehow those artifacts must have been generated in-source since they weren't generated in the action... 🤔

Yep! There is some magic asset stuff in git! https://github.com/jprendes/emception/commit/9f45133748f5e7d935f0e6dc43f60a5a34f6ce7a#diff-ba33a14dd8aef11abcbf35a3323ca63901264d15c73e69175e29f54606b082ae

Oh actually it looks like there's a whole branch that is being used as a github pages deployment drop point: https://github.com/jprendes/emception/tree/demo


How do we change the frame-shift for extracting features?

if you are talking about the WORLD vocoder then change the value here https://github.com/CSTR-Edinburgh/merlin/blob/master/tools/WORLD/test/analysis.cpp#L311 to desired value and you need to take care similarly in synthesis file https://github.com/CSTR-Edinburgh/merlin/blob/master/tools/WORLD/test/synth.cpp#L293. Generally, in speech synthesis people use 5ms as a frame-shift for extracting features.

Thanks bajibabu.
What about https://github.com/CSTR-Edinburgh/merlin/blob/master/tools/WORLD/test/synth.cpp#L49 ?


Add French support

This a nice thing to have but not required for 1.0.0 removing it from the scope

Done at https://github.com/MarckFp/ppoc-gen/commit/d1e802338f25425f0c07c050203fbe399294bc34 Thanks to Chat GPT translation features


Feature: add upload timestamp/integrity checksum

Bug Report
Check to see if a bug report already exists.  If you still want to create a new issue please include the information below.
Stack Trace or error message
Environment

Sia version:
OS:

Any useful logs
Feature Request
Check to see if the feature is on the roadmap.  If it is not on the roadmap then clearly describe your feature request and why it would benefit Sia.
Sia roadmap

Related to #2109
Recommendation to devs: either update the body of this issue to be more descriptive or close it as a duplicate of #2109.

Want to go ahead and include in this issue all common file metadata such as creation time, etc.

@MeijeSibbel @tbenz9 @DavidVorick how are you defining creation time versus upload timestamp?  Is the intent that upload timestamp is when the upload was initiated and the creation time is when the file has reached it's full redundancy?  Or is that a duplication in the request?

I was imagining Sia would capture the Linux atime, ctime, and mtime timestamp independent of the upload and last-download timestamps.
For example, I take a picture on 1/1/2018, rename that photo on 2/2/2018, view the file on 3/3/2018, upload it to Sia on 4/4/2018, download from Sia on 5/5/2018.  It would have a:
creation time of 1/1/2018
modify time of 2/2/2018
access time of 3/3/2018
upload time of 4/4/2018
last download time of 5/5/2018
My opinion is upload time should be when the file is actually submitted to Sia for upload, not necessarily when it reaches full redundancy. If we wanted to get fancy I guess we could have as many as 3 upload timestamps (submitted, 1x, 3x redundancy).

thanks @tbenz9
and I saw your commit on adding the upload and mod times, were you planning on continuing to finish those edits and submit a PR?  If not I can take what you started with and continue on from there.  LMK

I am not planning on finishing #2860.  Once I began to understand the scope of the PR I chickened out and decided I didn't have the skills to write the compatibility code for the on-disk .sia file changes.  Please feel free to pick up where I left off.
I'll add that if there is any other relevant metadata returned by stat for a file this would be an ideal time to add it.

One more piece of metadata I would love to see would be a sha1(or another algorithm) sum of the file before it's been uploaded to Sia, that hash should be stored in the .sia file and accessible to the user through the API/siac.
There are a couple use cases for this:
1: A user can check if the version of a file uploaded to Sia is the same version they have on their computer.
2: A user can manually confirm that a file uploaded, and downloaded from Sia is the same file.

Does it make sense to have atime be different from download time? Isn't a download an access?
I can see why you'd have a difference between creation and upload time though.

This should probably be done after the new .sia format #2204


Enable HTTPS

GitHub now has https support! Woo!
https://blog.github.com/2018-05-01-github-pages-custom-domains-https/
Looks like you need to remove and re-add the custom domain from the Pages section of repo settings to get it to request a cert (as it is at an apex, using A records). 😃

Also enforce https would be awesome!

Yep. Tried that this morning, but still waiting for my DNS changes to propagate :P

DNS looks good from here 👍

Tell that to Github? Unless I'm missing a step, I've removed and re-added the custom domain a half a dozen times today. Maybe I need to push something to kick it in the pants?

It verks! Its verking!
Not with www but this'll do :P


Linux package bot failing - pub crash in observatory build

With the new version of pub, the linux src package bot is failing.  There is a crash in pub during the observatory build:
https://uberchromegw.corp.google.com/i/client.dart/builders/linux-distribution-support-debian_wheezy-be/builds/7168/steps/Create src tarball/logs/stdio
Uncaught Error: Git error. Command: git rev-parse HEAD
fatal: Not a git repository (or any of the parent directories): .git
Stack Trace:
#0      runSync (package:pub/src/git.dart:75:24)
#1      PubCommandRunner._checkDepsSynced (package:pub/src/command_runner.dart:171:21)
#2      PubCommandRunner.runCommand.<runCommand_async_body> (package:pub/src/command_runner.dart:94:5)
#3      Future.Future.microtask.<anonymous closure> (dart:async/future.dart:144:37)
#4      _microtaskLoop (dart:async/schedule_microtask.dart:43:11)
#5      _microtaskLoopEntry (dart:async/schedule_microtask.dart:52:5)
#6      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:96:13)
#7      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:149:5)

#0      _rootHandleUncaughtError.<anonymous closure> (dart:async/zone.dart:895:5)
#1      _microtaskLoop (dart:async/schedule_microtask.dart:43:11)
#2      _microtaskLoopEntry (dart:async/schedule_microtask.dart:52:5)
#3      _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:96:13)
#4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:149:5)
make[2]: *** [runtime/observatory/pubspec.lock] Error 255

@nex3 @johnmccutchan

This looks like the code I added to make sure pub is running from the correct revision. Is there anything unusual about the build setup for this bot? Is pub not checked out from git, somehow?

That is correct, the debian packages are built from the source tarball, and
the whole point of a source tarball is that the package is compiled from
the files contained in the tarball, with no additional checkouts.  You have
to revert this check.  If you want, you could have a check that if pub is
checked out from git, then it is the correct revision.
We cannot release any new versions of Dart until this is reverted.
On Wed, Jul 15, 2015 at 9:28 PM, Natalie Weizenbaum <
<EMAIL_ADDRESS>wrote:

This looks like the code I added to make sure pub is running from the
correct revision. Is there anything unusual about the build setup for this
bot? Is pub not checked out from git, somehow?
—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/23846#issuecomment-121721133.

--
William Hesse

I think we should revert the change so that a new release can get built.
On Mon, Jul 20, 2015 at 7:27 AM, William Hesse<EMAIL_ADDRESS>wrote:

That is correct, the debian packages are built from the source tarball, and
the whole point of a source tarball is that the package is compiled from
the files contained in the tarball, with no additional checkouts. You have
to revert this check. If you want, you could have a check that if pub is
checked out from git, then it is the correct revision.
We cannot release any new versions of Dart until this is reverted.
On Wed, Jul 15, 2015 at 9:28 PM, Natalie Weizenbaum <
<EMAIL_ADDRESS>wrote:

This looks like the code I added to make sure pub is running from the
correct revision. Is there anything unusual about the build setup for
this
bot? Is pub not checked out from git, somehow?
—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/23846#issuecomment-121721133.

--
William Hesse
—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/23846#issuecomment-122902449.

I have https://codereview.chromium.org/1246713005 out to fix this.
On Mon, Jul 20, 2015 at 8:16 AM, Dan Grove<EMAIL_ADDRESS>wrote:

I think we should revert the change so that a new release can get built.
On Mon, Jul 20, 2015 at 7:27 AM, William Hesse<EMAIL_ADDRESS>wrote:

That is correct, the debian packages are built from the source tarball,
and
the whole point of a source tarball is that the package is compiled from
the files contained in the tarball, with no additional checkouts. You
have
to revert this check. If you want, you could have a check that if pub
is
checked out from git, then it is the correct revision.
We cannot release any new versions of Dart until this is reverted.
On Wed, Jul 15, 2015 at 9:28 PM, Natalie Weizenbaum <
<EMAIL_ADDRESS>wrote:

This looks like the code I added to make sure pub is running from the
correct revision. Is there anything unusual about the build setup for
this
bot? Is pub not checked out from git, somehow?
—
Reply to this email directly or view it on GitHub
<https://github.com/dart-lang/sdk/issues/23846#issuecomment-121721133
.

--
William Hesse
—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/23846#issuecomment-122902449.

—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/23846#issuecomment-122915456.

Natalie - is this bug still alive?

No, it was fixed in dart-lang/pub@8abe19766853ad58c4c107a73678e2bccfeb1d4b.


Create page for App Mesh adapter in Meshery Docs

Enhancement Description
A new placeholder page needs to be created to show upcoming support for App Mesh on https://meshery.layer5.io/docs/adapters. A new Maesh adapter page needs to be created.
Implementation

Create new page in adapters folder.
Add link to App Mesh adapter page in adapters table.

See instructions for contributing to Meshery docs - https://github.com/layer5io/meshery/blob/master/CONTRIBUTING.md#documentation-contribution-flow

Partially closed by https://github.com/layer5io/meshery/pull/263

Closed by https://github.com/layer5io/meshery/pull/261


Allow users to un-vote

After a user has voted, the user should be able to click on the same estimation card to remove their previous vote.

We're also absolutely missing this feature. It's not uncommon that during discussion about a work item, things come up, which lead to the fact that you want to un-vote and wait for the end of a discussion to vote again. We always used "if everyone has casted his vote, we can reveal all" rule which is now not working anymore.

This is now implemented - see linked commit.
Users can click their selected card again to "un-vote".


Add the document for failover overview

Signed-off-by: changzhen<EMAIL_ADDRESS>What type of PR is this?
/kind feature
/kind documentation
What this PR does / why we need it:
Add the document for failover overview.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:

/cc @RainbowMango @Poor12

Please @Poor12 take a look.

/lgtm
/cc @RainbowMango


Implement Queue Clients and Builders

Implement the Queue client and builder classes based on this https://gist.github.com/jianghaolu/7401f848a0354de19fcddc8bbfa5bd05#file-queue_spec-md

Rolled up into https://github.com/Azure/azure-sdk-for-java/issues/4037


How to run two containers using testcontainers

In my test class, i want to run two container for example, database container and application container. But Always it is running application container. here is my sample code.
public class Test
{
int hostPort = 3000;
int containerExposedPort = 3000;
Consumer cmd = e -> {
e.withPortBindings(new PortBinding(Ports.Binding.bindPort(hostPort),
new ExposedPort(containerExposedPort)));
e.withHostName("db-host");
e.withName("db-container-name");
};
Consumer<CreateContainerCmd> cmd1 = e -> {
    e.withPortBindings(new PortBinding(Ports.Binding.bindPort(8080),
            new ExposedPort(8080)));
    e.withHostName("app-host");
    e.withName("app-container-name");
};

@Rule
public GenericContainer db = new GenericContainer(
        "db:latest")
        .withExposedPorts(containerExposedPort)
        .withClasspathResourceMapping("db.conf",
                "db.conf", BindMode.READ_ONLY)
        .withCommand("--config-file db.conf")
        .withCreateContainerCmdModifier(cmd);

@Rule
public GenericContainer app = new GenericContainer(
        "app:latest")
        .withExposedPorts(8080)
        .withCreateContainerCmdModifier(cmd1);

@Test
public void test()
{
	System.out.println("asis");
}

}

Hi @Asisranjan,
Do you see any errors reported by TestContainers in this particular scenario? Would you be able to post that error message? I tried something similar to your code and it worked. Here is the working code:
public class TwoContainerTest {
Consumer cmd = e -> {
e.withHostName("db-host");
e.withName("db-container-name");
};
Consumer<CreateContainerCmd> cmd1 = e -> {
    e.withHostName("app-host");
    e.withName("app-container-name");
};

@Rule
public GenericContainer ol1 = new OpenLdapContainer()
        .withClasspathResourceMapping("db1.ldif",
                "/container/service/slapd/assets/config/bootstrap/ldif/50-bootstrap.ldif", BindMode.READ_WRITE)
        .withCommand("--copy-service")
        .withCreateContainerCmdModifier(cmd);

@Rule
public GenericContainer ol2 = new OpenLdapContainer()
        .withClasspathResourceMapping("db1.ldif",
                "/container/service/slapd/assets/config/bootstrap/ldif/50-bootstrap.ldif", BindMode.READ_WRITE)
        .withCommand("--copy-service")
        .withCreateContainerCmdModifier(cmd1);

@Test
public void test()
{
    System.out.println("asis");
}

}
Observation: I saw two containers named appropriately coming up in this case. I also saw some errors with port binding code initially so I removed that from this test code.

This is quite an old issue, unfortunately. @Asisranjan did you manage to resolve this?

Yes Richard. I followed as you suggested here

Hi Richard
There are couple of queries. I am mentioning here.

I am trying to test REST webservices. REST webservice based application running in Container. Observing my test cases running before my server started, hence my test case is failing. How to wait for the server ready before any test execution
I am setting up in Oracle Virtual Box, Since I am trying to get mapped port for consuming rest service and the same port is not forwarded by default, Hence the test case is failing. Is there any way to resolve this issue.

Thanks
Asisranjan Nayak


[MTM-56084]: adding new java client for mTLS protocol using X509 certificate

Documentation is available on :
https://sagportal.sharepoint.com/:w:/r/sites/IoTAnalyticsRnDOps/_layouts/15/doc2.aspx?sourcedoc={489B6AA8-3AFA-44DC-BD83-32DDC4B93F43}&file=Enable mTLS for REST.docx&action=default&mobileredirect=true
API documentation :
https://github.softwareag.com/IOTA/cumulocity-openapi/pull/234/

Closing this one as https://github.com/SoftwareAG/cumulocity-examples/pull/115 is merged


Document best practices for SESSION_COOKIE_AGE and SESSION_SAVE_EVERY_REQUEST for inactivity timeout

Proposed Changes
Some admins would like to have nautobot log users out after a period of inactivity. The proposed method is to set the SESSION_COOKIE_AGE to the desired timeout value and then use SESSION_SAVE_EVERY_REQUEST to have Django update the session on every request. However, nautobot uses the database session backend by default so making a database UPSERT on every request may impact performance. We should test the performance impact and document whether admins should also use the SESSION_ENGINE = "django.contrib.sessions.backends.cached_db" setting when setting SESSION_SAVE_EVERY_REQUEST = True.

Justification
Documentation

Any update regarding the above? Is enabling SESSION_COOKIE_AGE & SESSION_SAVE_EVERY_REQUEST compromising performance? And if yes, what is the best way to implement it?


Rename source-dir to 'api'

This makes it easier in gitea to see what is API-stuff and what isn't, without having to rename everything.

There is still package gitea inside of the files

@tboerger hehe, code-reviews are nice :trollface:

I think this PR is no need. Just keep the gitea name for identifier.

On Mon, Nov 07, 2016 at 06:42:24AM -0800, Lunny Xiao wrote:

I think this PR is no need. Just keep the gitea name for identifier.

+1


How do we know the different font exist in this library ?

Hi,
How do we know the list of font's supported by this library ?
I couldn't find any documentation related to supported font
Is uber like font [ ff clan ] is supported ?

@roshu10 please read the documentation, You add the fonts that you want to use into your assets folder.


Hard-code permalink values into posts at publish time

The permalink value of _config.yml might change, which would cause existing URLs to break. As a possible way around this, at post time: take the _config.yml permalink value, and store it as a header in the post file. This way, when _config.yml changes, existing posts don't break and all is right with the world.
To avoid breaking existing sites, when generating a published post, if it has no permalink header, read the value out of _config.yml and hard-code it. This could eventually be removed once it is likely that all users have migrated to having permalinks hard-coded. No manual intervention needed.
Question: Use, say, /:year/:title, or the explicit, /2013/my-blog-post in the file's permalink? I'd argue the latter, to be extra conservative, but this might go against expectations if the user:

Publishes a post.
Sees their slug was incorrect, e.g., my-pots instead of my-post.
Goes to edit the post on the filesystem manually so the slug is my-post.
Republishes.

They might expect the URL to fix itself, but it won't if the permalink is hard-coded. (This is largely a non-issue for those using exclusively the admin interface, where editing post URLs has no interface.)

Changing permalink is just going to have caveats, whatever you do.


Grid verticalAlign="middle" FOUC

I'm trying to create a center aligned FlexContainer wrapper component using the Grid component:
https://github.com/itsmoops/react-redux-webpack/blob/master/src/components/shared/flex-container.jsx
Steps
Create a height: 100% Grid with verticalAlign="middle"
Expected Result
Grid content should render in the center of the page
Actual Result
Content flashes on the top of the screen before rendering correctly in the center (only on initial page load)
Version
0.68.3
Testcase
This codepen does not seem to produce the issue
https://codepen.io/itsmoops/pen/WjWMxb
But I am seeing it in my boilerplate which (seemingly) is doing the same thing...
https://github.com/itsmoops/react-redux-webpack
npm i
npm start

Semantic UI React components simply render to valid Semantic UI HTML, there is no window of opportunity available to create a FOUC.  In order for this to happen, the CSS must be getting loaded after the HTML body is loaded.
It appears you are using the style-loader, which:

Adds CSS to the DOM by injecting a

@levithomason thank you for the thorough explanation, this was exactly the issue! I ended up adding extract-text-webpack-plugin to my webpack build process and referencing the generated file in the <head> as you suggested and boom, no more FOUC.


Missing tables in C.4 (and a typo)

C.4 contains “FIXME: Replace with first format strings table” and “FIXME: Replace with second format strings table”, oops!
These tables did exist and IIRC there was an issue with the Word translator? Anyway they didn’t make it into the final version.
There is also a small typo, IFormatable rather than IFormattable, which can be fixed alongside putting the tables back.

Thanks @Nigel-Ecma. I have restored the tables, exactly as they existed in V6. After some dialog with Ecma management, I've sent along to them a revised draft of V7 with the deleted placeholders and inserted tables marked using pseudo-tracked changes, which will be turned into ordinary text before publication..
I also fixed the typo, but only in the Word version. That still needs to be fixed in the md. As such, I'll leave this issue open until that fix has been made.


No sound for lightning

Currently, there is no sound for lightning.

Hello, I would like to contribute. I am new to GitHub and would like to know how to get sarted reproducing this bug. I would very much appreiate any feedback. Thanks in advance.

@pillsbury7901 If you haven't already, you should read this document: https://github.com/mc-server/MCServer/blob/master/CONTRIBUTING.md
The easiest way to reproduce this bug would be to start a server and change the weather to stormy, so that a lightning strike will occur every so often.

Sorry, wrong document. You should read that one too, but this one is more important for a newcomer to the project:
https://github.com/mc-server/MCServer/blob/master/GETTING-STARTED.md

Thank you for the response. I have been trying to compile MCServer for the past few days to no avail. Do you have any suggestions for compiling on a linux box? I believe the error is related to gcc but I have not been able to remedy this issue. Thanks again.

Its probably a gcc version that does not support c++11. Check gcc --version, and if its less than gcc 4.8 then you need to install a more recent gcc.

This is what i feared. When I attempt to update with apt-get or aptitude it says that I have the latest version installed. I have downloaded gcc-5.1.0 and followed the installation instructions from the Documentation precisely but I am still getting the same error.

If you've got multiple versions of gcc installed it may be that you're calling the wrong one. Try which gcc and if thats wrong, specify the path to the correct one when calling cmake, like CC=<path/to/gcc> CXX=<path/to/g++> cmake

Thank you. I will try this now.

Question, must I buy a Minecraft Client to debug and fix code for this project?

There are tasks that don't require any interaction with the client; as far as I know @worktycho isn't using a client at all. But of course majority of tasks will require some form of verification through the client interface. So it really depends on what you want to be doing.
This particular task requires a client to verification that a fix is indeed working.


Massive Scale Launches III - Add a to orbit bonus for non-Kerbin

Add a bonus scaled based on planet weight for reaching orbit from a landed/splashed state on bodies other than the homeworld.

This is silly, guess it's for EL?


Switch to polling strategy for cached contracts

Because updates to transformers, triggers and types are infrequent,
streaming these contracts generates a lot of overhead. Switching to
a slow polling strategy should reduce load on the DB.
This change also adds a stop() method that can safely shutdown the
worker, waiting for jobs to finish and stopping the polling.
Change-type: major
Signed-off-by: Lucian Buzzo<EMAIL_ADDRESS>

@balena-ci retest

Oh nice! I was wondering if there was a built in for this
Sent from my iPhone

On 9 Jun 2022, at 19:00, Page- @.***> wrote:

@Page- commented on this pull request.
In test/integration/index.spec.ts:

+const delay = (ms: number) => {

return new Promise((resolve) => setTimeout(resolve, ms));
+};
You can just import an equivalent directly from timers/promises

⬇ Suggested change
-const delay = (ms: number) => {

return new Promise((resolve) => setTimeout(resolve, ms));
-};
+import { setTimeout } from 'timers/promises';
or if you want to keep the name:

⬇ Suggested change
-const delay = (ms: number) => {

return new Promise((resolve) => setTimeout(resolve, ms));
-};
+import { setTimeout as delay } from 'timers/promises';
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Thanks for the tip @Page- I've updated the code to use timers/promises

@balena-ci retest


Cheery-pick Fix drone fail on photon tdnf call

Coverage remained the same at 64.311% when pulling 10c943c5badc6e0dd6bf4852e5977ce6ac41e7bd on wy65701436:fix-ci-1.5.0 into c96aef765e1adaad56d8dbbf215a8eb3474e23b6 on vmware:release-1.5.0.

please rebase.


🦺[TechDebt] - Update Support URL

Updated to new URL (New URL site is not yet live.)
Because:

We are switching the support URL to https://litewallet.io

@kcw-grunt What's our ETA on this site switching over to the new URL? Just whenever?

This is block until : https://github.com/litecoin-foundation/litewallet.io/issues/2 is merged.

Thanks for grabbling this @JasperHeist

I guess no reason not to get this into develop if the change is imminent. Let me know

@JasperHeist . I made a mistake and thought this would be needed for the support URL. Since then I got Cloudflare access where I can re-direct .  So this is not needed.
This small update is needed though: #89


Replace vast::directory with std::filesystem in segment_store test

:notebook_with_decorative_cover: Description

Problem:

segment_store unit tests uses vast::directory which is going away
soon.

Solution:

Replace vast::directory uses with std::filesystem.
Replace vast::path uses with std::filesystem::path.

:memo: Checklist

[ ] All user-facing changes have changelog entries.
[ ] The changes are reflected on docs.tenzir.com/vast, if necessary.
[ ] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions

File-by-file.

The AppleClang CI job seems to have spuriously failed for this PR but it should be unrelated to these changes. This PR only addresses purely only one specific unit test in isolation. Note the other CI jobs passed just fine @dominiklohmann

The AppleClang CI job seems to have spuriously failed for this PR but it should be unrelated to these changes

We sometimes have repeated connection failures between server and client in CI, which we can never reproduce locally. I suspect it's something with GitHub's networking setup. We usually just re-run these.


Wires - Dragging a connection causes the screen to get blank, all shapes disappear

Consider a wires connector with 2 connections, and so only 2 control points, and consider both connections linked to a shape.
In this case, if dragging one of the connections over one of the other side,  causes all shapes to disappear, the canvas gets blank.

Hey @danielzhe ! no worries, yeah I'm almost sure we've already fixed this one, IIRC it was due to some NPE... so I'll proceed to closing it, we're always on time to reopen... :)
FYI - I'll do an update to this repository code and do a review to the issues during next few days... as it's  a bit "out-dated" sorry. This is because we're working right now (just temporary!!) on a fork here, so some new features and fixes are being included in our fork but not yet "back-ported" here. Will do during next few days, sorry for the inconveniences...
Thanks again for your collaboration!

Closing - out of date


CPUID,Inc.CPUIDpowerMAX version 1.00

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Keep it CPUID instead of CPUID,Inc and please don't add comma in PackageIdentifier


execution policy issue with profile

If you get something like

profile.ps1 is not digitally signed. You cannot run this script on the current system.

You probably need to change your execution policy. I will however not recommend any setting for this. Perform a search for Set-ExecutionPolicy.

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d3cbeb9a-9e7c-f8f4-5c03-5d6262d2b440
Version Independent ID: eb575c79-5b2a-45f6-646f-9d41d5d223e8
Content: Windows Terminal Powerline Setup
Content Source: TerminalDocs/tutorials/powerline-setup.md
Product: dev-environment
Technology: windows-terminal
GitHub Login: @cinnamon-msft
Microsoft Alias: cinnamon

Looks like this was fixed in #240.


Is this project still alive?

I really appreciate the effort you put in CalcBinding and i'm aware of the amount of work it is needed to keep a project going.
By the way pull requests are accumulating and new issues are not even being discussed.
If you don't have the time or inclination to support this project anymore, please consider to transfer the project to someone else.
We really need a few fixes :)
Hope to hear you
Thank you

Hello, maurosampietro
Thank you for your opinion on the project. I really paused work on the project, because solving of many issues and new devemoplent requires very deep analysis of nondocumented WPF part, or is impossible because of inextensibility of the WPF architecture.
I will look at issues list again and analyse which of the problems can be solved. Also I will write review about pull requests.


Alternative to quick start's code snippet?

Environment details
python 3.8.3
google-cloud==0.34.0
Issue
Is there a way I can pass the SCOPES, access token( or creds) and json file without calling
flow = InstalledAppFlow.from_client_secrets_file(
                'credentials.json', SCOPES)
            creds = flow.run_local_server(port=0)

?
As i can't run a local server on an Alexa skill. At the same time, the module  doesn't have an arg for the .json file to be inputted. if I were to use instead.

If you are accessing resources on behalf of a user, you need to obtain access tokens. run_local_server is a method provided to make that convenient when developing locally. If using that is not possible, you'll have to obtain the access and refresh tokens some other way. See https://developers.google.com/identity/protocols/oauth2.
You would then create Credentials objects using the class constructor.
If you're not accessing the API on behalf of user account, you can use a service account instead.
https://cloud.google.com/docs/authentication/production

Hi @DaniyalDhz,
I'm going to close this issue as guidance was provided in the last comment. If you're still having trouble please re-open this issue with additional information.


Feature static link lib download script

add one function on downloading different files according to user's OS

some directories:

Linux: static library downloads to deps/llibrocketmq.a
macOS: dynamic library downloads to deps/librocketmq.dylib
Windows: dynamic and static libraries download to deps/rocketmq-client-cpp.{dll,lib}

/cc @duanpengfei

btw, we can use urllib's streaming feature to write a stream to a file stream.


[BUG] Bullet points not visible

Outline bullet points are not visible.
I have:
Restarted Obsidian
Restarted the Mac
Uninstalled and reinstalled the plug-in with the current version 4.7.1
Uninstalled and reinstalled Obsidian
Environment (please complete the following information):
OS: macOS 12.6.3 (Monterey)
Obsidian version: v1.3.4
Plugin Version: 4.7.1
Additional context
M1 Max Macbook Pro

I tried installing the plugin inside a fresh VM and it just doesn’t work. Is this still being actively developed?

I think you first have to create the bullet list with -.


dm table-routing need support regular expressions

demand:
db list: test
table list:
t_1
t_2
t_11
t_12

now,table routing need write:
routes:
  r1:
    s-p: 'test'
    t-p: 't_[0-9]'
    t-s: 'test'
    t-t: 't'
  r2:
    s-p: 'test'
    t-p: 't_[0-9][0-9]'
    t-s: 'test'
    t-t: 't'
  r3:
    t-s: 'test'
    s-p: 'test'

i wish:
routes:
  r1:
    s-p: 'test'
    t-p: '~t_*'
    t-s: 'test'
    t-t: 't'

include version:
v2.x
v1.x
v5.3.0

/assign buchuitoudegou

you can use wildcard matching to get the same effects, check this:
routes:                           # The routing mapping rule set between the data source tables and downstream TiDB tables. You can set multiple rules at the same time.
  route-rule-1:                   # The name of the routing mapping rule.
    schema-pattern: "test"      # The pattern of the upstream schema name. Wildcard characters (*?) are supported.
    table-pattern: "t_*"          # The pattern of the upstream table name. Wildcard characters (*?) are supported.
    target-schema: "test"         # The name of the downstream TiDB schema.
    target-table: "t"             # The name of the downstream TiDB table.

Technical Design of the New Router
The new router reuses the filter module in the tidb-tools project, which is able to recognize regular expressions and matches them against input strings. The filter has also been part of the block-allow-list and thus, reusing the filter module ensures consistency amongst all the components.


Poor WiFi AP performance on Rev 1 chips on specific channels

I am observing poor AP performance on Rev 1 chips, when the channel is set to 2, 4 or 5. The same code operates well on Rev 0 chips. The WiFi environment on those channels is relatively free from interference.
Wireshark shows occasional TCP re-transmissions, but additional testing showed TCP and UDP to be equally effected.
Currently building with esp-idf commit cf2600a
Edit: A co-worker has confirmed that his Rev 1 chips do not show the same behavior. If it is indeed the WiFi environment causing the issue, why are Rev 0 chips not effected?

Did you try to erase flash or force full calibration?

I have performed make erase_flash. Is there another method of triggering calibration?

There is a menuconfig option to not store calibration so it will be performed every boot

Additional testing has found that this problem only occurs in some of my modules. Is it likely a result of their installation, and not an underlying issue with the Rev 1 chip.
Apologies, I jumped the gun on this one.


[Android] Add formatting actions to compose library

Changes
Add formatting actions to the Android compose library by exposing new methods on RichTextEditorState:

Inline formatting (bold, italic, underline, strike-through, code)
Lists
Code blocks
Quotes

Also

Expose undo / redo actions
Add everything to the example app

Screenshot

Context

Part of https://github.com/matrix-org/matrix-rich-text-editor/issues/315

Codecov Report
Patch coverage has no change and project coverage change: +1.57% :tada:

Comparison is base (2621d6e) 88.26% compared to head (afb53df) 89.83%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #755      +/-   ##
============================================
+ Coverage     88.26%   89.83%   +1.57%
============================================
  Files           147       82      -65
  Lines         17454    14693    -2761
  Branches        783        0     -783
============================================
- Hits          15405    13200    -2205
+ Misses         1862     1493     -369
+ Partials        187        0     -187

Flag
Coverage Δ

uitests
?

uitests-android
?

uitests-ios
?

unittests
89.83% <ø> (+1.73%)
:arrow_up:

unittests-android
?

unittests-ios
?

unittests-rust
89.83% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 65 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


ttylog: 0.26 -> 0.29

Motivation for this change
Update
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

If you are referring to the version. It's personal preference. I always bracket variables
Sent from my iphone
On Mar 2, 2017, 7:13 PM -0600, Tobias Geerinckx-Rice<EMAIL_ADDRESS>wrote:

@nckx commented on this pull request.
In pkgs/tools/misc/ttylog/default.nix (https://github.com/NixOS/nixpkgs/pull/23417#discussion_r104070097):

src = fetchFromGitHub { owner = "rocasa"; repo = "ttylog"; - rev = version; - sha256 = "1xvqch1dwpl2d89n9cjcxhqbpvm50fspj5yn883wbii6dpg62p84"; + rev = "${version}";

Why this change?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/NixOS/nixpkgs/pull/23417#pullrequestreview-24885453), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAHL73H5uns2WvZZbAPd2UAJfGEftjc8ks5rh2kdgaJpZM4MRvVh).

Applied in 55e376078a, thank you!