AspNetCore.Correlation.Google cookie not found

After upgrading from 3.0 to 3.1 I have issues with the Google authentication. I have no cookie policy registered and this is how my authentication configuration looks like:
services
          .AddAuthentication(options =>
          {
            options.DefaultScheme = IdentityConstants.ApplicationScheme;
            options.DefaultSignInScheme = IdentityConstants.ExternalScheme;
          })
          .AddGoogle(options =>
          {
            options.ClientId = _configurationOptions.Google.ClientId;
            options.ClientSecret = _configurationOptions.Google.ClientSecret;
            options.Scope.Add("https://www.googleapis.com/auth/userinfo.profile");
            options.SaveTokens = true;
          })
        .AddIdentityCookies(options =>
        {
          options.ApplicationCookie.Configure(a =>
          {
            a.Cookie.Name = ".AspNet.SharedCookie";
            a.Cookie.Domain = _configurationOptions.Domain;
          });
        });

And this is the error I get:
warn: Microsoft.AspNetCore.Authentication.Google.GoogleHandler[15]
      '.AspNetCore.Correlation.Google.611LD72jlzagvpKikBe8B41IqHTqrvVnXZdWS9iuZd8' cookie not found.
fail: Microsoft.AspNetCore.Server.Kestrel[13]
      Connection id "0HLRR2LP0DVR7", Request id "0HLRR2LP0DVR7:00000001": An unhandled exception was thrown by the application.
System.Exception: An error was encountered while handling the remote login.
 ---> System.Exception: Correlation failed.
   --- End of inner exception stack trace ---
   at Microsoft.AspNetCore.Authentication.RemoteAuthenticationHandler`1.HandleRequestAsync()
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpProtocol.ProcessRequests[TContext](IHttpApplication`1 application)```

What browser are you using? Have you implemented the changes needed to support older browsers?

Sharing a decrypted Fiddler trace is the best way to debug this class of issue.

@blowdart I use Chrome 78 and I have not implemented any support for older browsers.

As Chris says a fiddler trace would be highly useful

Ok. I use macOS but I could try the mono version of Fiddler maybe

Does it happen if a PC is hitting the web site? Or is this just in your local development?

Another way to capture a raw trace is with Kestrel's connection logger.
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/servers/kestrel?view=aspnetcore-3.1#connection-logging

@blowdart I have only tested this on my mac running in docker. If I just tested on my mac and it seems to work fine. So it seems to be only when I run the application in Docker.

Oh then it's probably how keys are persisted. Does it happen when the container restarts but you still have the browser open?

@blowdart I just tested from a Windows machine running in parallels and worked just fine. So it seems to be when I access the application from chrome from my Mac. Firefox on my Mac works fine. Running the application locally works fine in all browsers so it seems to be a combination of running in docker and access the app from my Mac using chrome

And the app is up all the time? There's no container restart? And you've cleared cookies and cache? And nothing in the logs?

Again, a network trace is one or a kestrel connection trace (linked above) is the first step to debug this.

@Tratcher Ok, I see. Where is the log file / dump stored when I configure the connection logging like this?
                    webBuilder.ConfigureKestrel(serverOptions =>
                    {
                        serverOptions.Listen(IPAddress.Any, 80, listenOptions =>
                        {
                            listenOptions.UseConnectionLogging();
                        });
                    });

@marcuslindblom it's in the application logs themselves (Console output by default, but you can configure other loggers). A network trace using a tool like Fiddler is even better.

@anurse My web and api runs using https with external auth so I guess that fiddler can not be used to read the packages. And on macOS is't a bit problematic to use Fiddler but wireshark works but I'm not familiar with http debugging using Wireshark. When using this config.
            Host.CreateDefaultBuilder(args)
                .ConfigureWebHostDefaults(webBuilder =>
                {
                    webBuilder.ConfigureLogging((_, factory) =>
                    {
                        factory.SetMinimumLevel(LogLevel.Debug);
                        factory.AddConsole();                    });
                    webBuilder.ConfigureKestrel(serverOptions =>
                    {
                        serverOptions.Listen(IPAddress.Any, 80, listenOptions =>
                        {
                            listenOptions.UseConnectionLogging();
                        });
                    });
                    webBuilder.UseStartup<Startup>();
                });

this is the only output in the console and I guess this is not what you are looking for...
warn: Microsoft.AspNetCore.Authentication.Google.GoogleHandler[15]
      '.AspNetCore.Correlation.Google.Lwev0vfdTrLb0UiyvMYc-Kre7ov5pdt77YpUdj_juVY' cookie not found.
fail: Microsoft.AspNetCore.Server.Kestrel[13]
      Connection id "0HLRVK8G34BJ0", Request id "0HLRVK8G34BJ0:00000001": An unhandled exception was thrown by the application.
System.Exception: An error was encountered while handling the remote login.
 ---> System.Exception: Correlation failed.
   --- End of inner exception stack trace ---
   at Microsoft.AspNetCore.Authentication.RemoteAuthenticationHandler`1.HandleRequestAsync()
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpProtocol.ProcessRequests[TContext](IHttpApplication`1 application)

Fiddler does have an option to decrypt https.
For UseConnectionLogging you'll need to turn the log level up to Trace.

Yeah, sorry I wasn't clear there, you can configure Fiddler to decrypt HTTPS.
Can you try using that or the connection logging (with the log level turned up to Trace as @Tratcher indicated)?

Here is a Trace from my container when the error occurs.
trace.log

Ah, you're using a proxy but it's not properly configured. The proxy is terminating https so the back-end app thinks the requests are using http. That causes the app to A) generate incorrect directs to http, and B) fail to mark cookies as 'Secure'.
See this doc about setting up forwarders for nginx and then adding UseForwardedHeaders to consume them.

Ok. Does this has to do with any recent upgrade because I have used this configuration for the past six months running in AWS without any problems. Also why is this only happening in chrome on my mac?

We've been dealing with issues related to SameSite cookies for the last several releases. It's particularly bad on Mac and iOS, and Chrome is now making it worse. There are a lot of moving parts here, but I suspect if you get your Secure attribute fixed that should take care of it.
Background:
https://github.com/aspnet/Announcements/issues/318
https://github.com/aspnet/Announcements/issues/348
https://github.com/aspnet/Announcements/issues/390
https://docs.microsoft.com/en-us/aspnet/core/security/samesite?view=aspnetcore-3.1
https://blog.chromium.org/2019/10/developers-get-ready-for-new.html

@Tratcher Ok I see. It does not seem to be easy to find the correct configuration and I can not make it work when I use exakt the configuration in the documentation. Maybe you could look at the config I ended up with.
My nginx configuration is located in /etc/nginx/conf.d/default.conf and the content is looks like this.
upstream api_server {
    server api;
}

upstream web_server {
    server web;
}

server {
    listen   80 default_server;
    # listen [::]:80 default_server deferred;
    return   444;
}

server {
    listen 80;
    listen [::]:80;
    server_name api.brickpile.local.computer;

    location /.well-known/acme-challenge/ {
        root /var/www/certbot;
    }

    location / {
        return 301 https://$host$request_uri;
    }
}

server {
    listen 80;
    listen [::]:80;

    server_name brickpile.local.computer;

    location /.well-known/acme-challenge/ {
        root /var/www/certbot;
    }

    location / {
        return 301 https://$host$request_uri;
    }
}

server {
    listen 443 ssl http2;
    listen [::]:443 ssl http2;

    server_name api.brickpile.local.computer;

    ssl_certificate         /etc/letsencrypt/live/brickpile.local/local.computer.crt;
    ssl_certificate_key     /etc/letsencrypt/live/brickpile.local/local.computer.key;

    include /etc/letsencrypt/options-ssl-nginx.conf;
    ssl_dhparam /etc/letsencrypt/ssl-dhparams.pem;

    location / {
        proxy_pass  http://api_server;
        proxy_http_version 1.1;
        proxy_set_header   Upgrade $http_upgrade;
        proxy_set_header   Connection keep-alive;
        proxy_set_header   Host $host;
        proxy_cache_bypass $http_upgrade;
        proxy_set_header   X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header   X-Forwarded-Proto $scheme;
    }
}

server {
    listen 443 ssl http2;
    listen [::]:443 ssl http2;

    server_name brickpile.local.computer;

    http2_push_preload on;

    add_header Set-Cookie "session=1";
    add_header Link $resources;

    ssl_certificate         /etc/letsencrypt/live/brickpile.local/local.computer.crt;
    ssl_certificate_key     /etc/letsencrypt/live/brickpile.local/local.computer.key;

    include /etc/letsencrypt/options-ssl-nginx.conf;
    ssl_dhparam /etc/letsencrypt/ssl-dhparams.pem;

    location / {
        proxy_pass  http://web_server;
        proxy_http_version 1.1;
        proxy_set_header   Upgrade $http_upgrade;
        proxy_set_header   Connection keep-alive;
        proxy_set_header   Host $host;
        proxy_cache_bypass $http_upgrade;
        proxy_set_header   X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header   X-Forwarded-Proto $scheme;
    }
}

map $http_cookie $resources {
    "~*session=1" "";
    default "</static/main.css>; as=style; rel=preload, </static/other.css>; as=style; rel=preload, </fonts/inter.css>; as=style; rel=preload, </fonts/Inter-upright.var.woff2>; as=font; crossorigin; rel=preload";
}

gzip off;

This is how my ConfigureServices looks like for the API.
    public void ConfigureServices(IServiceCollection services)
    {

      var dataprotection = services.AddDataProtection()
          .SetApplicationName("SharedCookieApp")
          .PersistKeysToFileSystem(new DirectoryInfo(_configurationOptions.DataProtection));

      services
          .AddAuthentication(options =>
          {
            options.DefaultScheme = IdentityConstants.ApplicationScheme;
            options.DefaultSignInScheme = IdentityConstants.ExternalScheme;
          })
          .AddGoogle(options =>
          {
            options.ClientId = _configurationOptions.Google.ClientId;
            options.ClientSecret = _configurationOptions.Google.ClientSecret;
            options.Scope.Add("https://www.googleapis.com/auth/userinfo.profile");
            options.SaveTokens = true;
          })
            .AddJwtBearer(options =>
            {
              options.Audience = JwtParameters.Audience;
              options.ClaimsIssuer = JwtParameters.Issuer;
              options.TokenValidationParameters = new TokenValidationParameters
              {
                ValidateIssuerSigningKey = true,
                IssuerSigningKey = new SymmetricSecurityKey(JwtParameters.SigningKey),
                ValidateIssuer = false,
                ValidateAudience = true
              };
            })
        .AddIdentityCookies(options =>
        {
          options.ApplicationCookie.Configure(a =>
          {
            a.Cookie.Name = ".AspNet.SharedCookie";
            a.Cookie.Domain = _configurationOptions.Domain;
          });
        });

      ConfigureRavenDbDocStore(services);

      services.AddIdentityCore<User>(options =>
      {
        options.Password.RequireDigit = true;
        options.Password.RequiredLength = 8;
        options.Password.RequireNonAlphanumeric = false;
        options.Password.RequireUppercase = true;
        options.Password.RequireLowercase = true;
      })
      .AddRoles<Raven.Identity.IdentityRole>()
      .AddRoleStore<RoleStore<Raven.Identity.IdentityRole>>()
      .AddUserStore<UserStore<User, Raven.Identity.IdentityRole>>()
      .AddSignInManager();

      services.AddControllers();

      services.AddCors(options =>
      {
        options.AddPolicy("AllowOrigin", builder =>
        {
          builder.AllowAnyMethod();
          builder.AllowAnyHeader();
          builder.WithOrigins(
            "https://brickpile.local.computer",
            "https://brickpile.app",
            "https://localhost:5003");
          builder.AllowCredentials();
        });
      });

      services.Configure<RouteOptions>(options =>
      {
        options.LowercaseUrls = true;
        options.AppendTrailingSlash = true;
      });

      services.AddApiVersioning(options =>
      {
        options.DefaultApiVersion = ApiVersion.Parse("1");
        options.AssumeDefaultVersionWhenUnspecified = true;
        options.ReportApiVersions = true;
        options.ApiVersionReader = new AcceptHeaderApiVersionReader();
      });

      services.AddSwaggerGen(c =>
      {
        c.SwaggerDoc("v1", new OpenApiInfo { Title = "My API", Version = "v1" });
      });

      services.AddHealthChecks();

and this is my Configure section for the API
    public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
    {
      if (env.IsDevelopment())
      {
        app.UseDeveloperExceptionPage();
      }

      app.UseForwardedHeaders(new ForwardedHeadersOptions
      {
          ForwardedHeaders = ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto
      });

      app.UseRouting();

      app.UseCors("AllowOrigin");

      app.UseCookiePolicy(); // Before UseAuthentication or anything else that writes cookies.
      app.UseAuthentication();
      app.UseAuthorization();

      app.UseSwagger();

      app.UseSwaggerUI(c =>
      {
        c.SwaggerEndpoint("/swagger/v1/swagger.json", "My API V1");
      });

      app.UseEndpoints(endpoints =>
      {
        endpoints.MapHealthChecks("/health", new HealthCheckOptions {
          ResponseWriter = WriteResponse
        });
        endpoints.MapControllers();
      });
    }

For what I can see the configuration looks correct but maybe some of my other settings interfere with ForwardedHeaders or something?

Check the troubleshooting section at the bottom.
https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/proxy-load-balancer?view=aspnetcore-3.1#troubleshoot
UseForwardedHeaders is a strict about who it accepts forwarders from. See the KnownProxies option.

I just tried to print all headers with this setting on both my sites.
      services.Configure<ForwardedHeadersOptions>(options =>
      {
        options.KnownNetworks.Clear();
        options.KnownProxies.Clear();
      });

      app.UseForwardedHeaders(new ForwardedHeadersOptions
      {
          ForwardedHeaders = ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto
      });

and this is the result:
Web
Request Method: GET
Request Scheme: http
Request Path: /home/index
Request Headers:
Cache-Control: no-cache
Connection: keep-alive
Pragma: no-cache
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.9,sv;q=0.8
Cookie: session=1; session=1
Host: brickpile.local.computer
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.79 Safari/537.36
DNT: 1
Upgrade-Insecure-Requests: 1
X-Forwarded-For: <IP_ADDRESS>
X-Forwarded-Proto: https
sec-fetch-dest: empty
sec-fetch-site: same-origin
sec-fetch-mode: same-origin

Request RemoteIp: ::ffff:<IP_ADDRESS>

API
Request Method: GET
Request Scheme: http
Request Path: /
Request Headers:
Connection: keep-alive
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9
Accept-Encoding: gzip, deflate, br
Accept-Language: en-US,en;q=0.9,sv;q=0.8
Host: api.brickpile.local.computer
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.79 Safari/537.36
DNT: 1
Upgrade-Insecure-Requests: 1
X-Forwarded-For: <IP_ADDRESS>
X-Forwarded-Proto: https
sec-ch-ua: Google Chrome 79
sec-fetch-dest: document
sec-fetch-user: ?1
sec-origin-policy: 0
sec-fetch-site: none
sec-fetch-mode: navigate

Request RemoteIp: ::ffff:<IP_ADDRESS>

The IP of my nginx container using this command docker inspect -f '{{range .NetworkSettings.Networks}}{{.IPAddress}}{{end}}' 76f9e098dcf2 is <IP_ADDRESS>.
But I still get the same error?

UseForwardedHeaders isn't running, it would have removed the x-forwarded-headers when it applied them. The app logs should say why.

Ok. I finally solved it and this is what I did.
I configured the default network in my docker compose file like this:
networks:
  default:
    ipam:
      config:
        - subnet: <IP_ADDRESS>/24

Configured forward headers like this:
    public void ConfigureServices(IServiceCollection services)
    {

      services.Configure<ForwardedHeadersOptions>(options =>
      {
        options.ForwardedHeaders = ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto;
        options.KnownNetworks.Add(new IPNetwork(IPAddress.Parse("::ffff:<IP_ADDRESS>"), 120));
      });
    ...
}

And add UseForwardHeaders like this:
    public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
    {
      if (env.IsDevelopment())
      {
        app.UseDeveloperExceptionPage();
      }

      app.UseForwardedHeaders();
    ....
}

Now it seems to work properly.
Thanks for your patience and help 👍

new IPNetwork(IPAddress.Parse("<IP_ADDRESS>"), 24) should also work in 3.0+.

Closing, as this appears to be resolved. Let us know if that's not the case and we can continue investigating!


0.28

It's already time for 0.28 thanks to our super short release cycles. Now, it' official...we hit 4000 stars on Github. That's amazing. Sorry, not as amazing as all the stuff that was going on for 0.27 but still pretty awesome.
Reload automation rules and groups
This release brings you a huge improvement of the automation and group handling. Both can be reloaded without a Home Assistant restart. The automations can be controlled directly from the frontend.

Climate and  cover
There are countless bugfixes included in this release which will make your experience with the climate and the cover platforms better. Two week ago was the biggest merger of implementations released that ever happened in the history of Home Assistant. Thanks to @turbokongen, @pvizeli, @djbanks, @danielperna84, and others the improvements on the code and the frontend side is continuing...
API documentation
The Home Assistant API Documentation is a great addition to the already exisiting user documentation. The focus is not end-users but developers who whant to get details about the code without actually browsing the code on Github.
Configuration validation
The validation of the configuration is still on-going. Approximatly 80 % is done. This means that we will propably talk about this topic in the next release notes again. To align the configuration of components and platforms we needed to break some. Please refer to the Breaking changes section to check if you need to update your configuration or simple check your log for configuration validation errors. Thanks to @kellerza, @fabaff, @Teagan42, and @pvizeli for your effort!
All changes

Light: Added bitfield to Flux LED since we are supporting effects (@tchellomello)
Device tracker: Owntracks waypoint import (@pavoni)
Climate: A lot of bugfix (@turbokongen, @pvizeli, @djbanks)
Notify: Improvement of the title handling (@lwis)
RFXtrx: Sensor cleanup (@turbokongen)
Device tracker: Fix for BLE device tracker (@open-homeautomation)
Device tracker: Allow 'None' MAC addresses to be loaded from known_devices file (@kellerza)
Cover: Bugfixes (@turbokongen, @danielperna84)
Sensor: Support for displaying details about crypto currencies (@fabaff)
Device tracker: Support for automatic to track your vehicles (@Teagan42)
Devie tracker: Add exclude option to Nmap device tracker ([danieljkemp])
Device tracker: Improved login errors for Asus device tracker (@kellerza)
Sensor: Support for displaying the status of Xbox Live accounts (@mKerix)
Notify: Adding link_names for sending Slack message (@salt-lick)
Binary sensor: Add the occupancy sensor class (@robbiet480)
Fan: Add supoort for MQTT fans (@robbiet480)
Docs: Add Sphinx API doc generation (@bbangert)
Core: Allow reloading automation without restart (@balloob)
Sensor: Added scale and offset to the Temper sensor (@mKerix)
Sensor: New support for [Trend] sensor (@pavoni)
Device tracker: Keep looking for new BLE devices (@Bart274)
Switch: Added device state attributes and support for legacy firmware for D-Link switches (@LinuxChristian)
Sensor: Improve 1-Wire device family detection (@Ardetus)
Modbus: Update to be thread safe (@persandstrom)
Camera: FFMpeg is abale to get the images (@pvizeli)
Core: Reload groups without restart (@balloob)
Core: Fix remove listener (@balloob)
Sensor: Support for monitoring your battery on a Linux host (@fabaff)
Core: Add support for complex template structures to data_template (@pvizeli)
check_config: Improve yaml fault tolerance and handle border cases (@kellerza)
Core: Add additional template for custom date formats (@lwis)
Sensor: Support for getting stats from Pi-Hole systems (@fabaff)

Breaking changes

OpenweatherMap entity IDs are now like sensor.owm_temperature. Previously they were like sensor.weather_temperature. Apologies for this change, but we needed to make OpenWeatherMap more generic now that we have many weather platforms.
Updates of configuration variables due to configuration check or alignment with other platforms. Please update your configuration entries according to the documentation:

OctoPrint component
mFi platform (switch and sensor)
NX584 Alarm Control Panel
Mediaplayer platforms: FireTV and MPD
switch and the cover command_line platforms

Release blog post


Updated MessageDispatcher usage

Updates accordingly to https://github.com/knative/eventing/pull/2847
Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>

Something is broken in nats dependency tree, i'm trying to find the root cause

Ok I've found the problem: the version of github.com/nats-io/nats-streaming-server (used only one time here in unit tests) clashes with the version of stan brought in by the latest cloudevents/sdk-go version. nats-streaming-server transitively depends on raft, same for sdk-go, but on two different versions: 1.1.1 and 1.1.2. Between these two there is a breaking change that breaks nats-streaming-server. I can't update nats-streaming-server because it depends on nats server 2 which uses go mods and go mod convention v2, which dep doesn't understand (https://github.com/golang/dep/issues/1962#issuecomment-540079542).
So i'm going to comment this test and replace with a dummy one

@slinkydeveloper rebase master, stanutil_test.go has been already removed. https://github.com/knative/eventing-contrib/tree/master/natss/pkg/stanutil

@pierDipi already rebased, but the go coverage blames me that i don't cover the stanutil... Anyway I've added a dummy test, let's see if this does the job

/retest pull-knative-eventing-contrib-integration-tests

/lgtm
/Approve
Thanks!!!!

/retest

Applying this patch https://github.com/knative/eventing/pull/2897 fixes everything except the TestBrokerChannelFlow/TestBrokerChannelFlow-{NatssChannel_messaging.knative.dev/v1alpha1}

@slinkydeveloper   it says
run hack/update-codegen.sh

did you do that ?

/lgtm
/approve


Adding CONTRIBUTION.md

Fixes #3372
Adding CONTRIBUTION.md that describes the workflow for the contributors at more details.

I like to amend the document.
Closing the PR.


You blocked www.wowhead.com

; <<>> DiG 9.10.3-P4-Ubuntu <<>> www.wowhead.com
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 63608
;; flags: qr rd ra; QUERY: 1, ANSWER: 3, AUTHORITY: 1, ADDITIONAL: 1
;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 4096
;; QUESTION SECTION:
;www.wowhead.com.               IN      A
;; ANSWER SECTION:
www.wowhead.com.        288     IN      CNAME   a005.zamnetwork.inscname.net.
a005.zamnetwork.inscname.net. 3600 IN   CNAME   inscname.net.
inscname.net.           3600    IN      A       <IP_ADDRESS>

tracking over at lightswitch05/hosts#11 - please close this one @daniel-widrick


Update node dependencies

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

is there a command you used to update these? Do you mind mentioning it in the PR message for archeological purposes / the next time someone needs to update the deps?


[Feature] Add configs of LSGAN

Add benchmark and configs for lsgan.
Refine some configs of styleganv2.

Codecov Report

Merging #14 (77cd8a4) into master (d7eda0b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   77.77%   77.77%
=======================================
  Files         103      103
  Lines        6070     6070
  Branches     1044     1044
=======================================
  Hits         4721     4721
  Misses       1068     1068
  Partials      281      281

Flag
Coverage Δ

unittests
77.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7eda0b...77cd8a4. Read the comment docs.


[move-function-value] Make sure that we propagate the debug scope from our original debug value into new undef debug values.

Otherwise if the move is in a different scope, the backend will think that we
have two different variables and emit the corresponding dwarf.


Restart specific debugger using command

Hi there.
A bit of background for the setup: I'm working with an Electron application, which requires two different debugging configurations - one for the main process, one for the renderer. I set up my launch.json so that the chrome attach automatically runs when the node launch is done starting (the same could also be achieved through a compound configuration).
But this setup has a major inconvenience: Whenever a task, extension or the user runs the "Debug: Restart" command, it restarts the configuration that is currently selected in the debugging widget (the one with the pause, continue and step buttons) or the "Debug Console" tab. By default, that's the main process launcher. Unfortunately, in 99% of cases the intended behavior is to restart the renderer, not the launcher, which means that the wrong debugger is restarted if the user doesn't pay attention to the currently selected one.
What I want to do is to configure a preferred debugger that will be restarted when the workbench.action.debug.restart command is invoked.

One possible solution would be to add a launch.json option for runtime priority so that the renderer can be selected, e.g. through a new presentation property (or by respecting the existing order option).
Another solution would be a parameter for the command, although I am not sure how that would interact with calling it from a task (using the ${command:<command>} syntax).

I am, of course, open for alternative solutions.

Possibly this extension could help: Launch Debug  and assign a keybinding to your renderer debug.

Possibly this extension could help: Launch Debug  and assign a keybinding to your renderer debug.

@DJ4ddi could you give Arturo's extension a try?

@DJ4ddi could you give Arturo's extension a try?

Unfortunately the extension does not solve my issue. I like the approach (defining a command for a specific configuration), but the problem is that it always attempts to start the configuration. In my case, I need it to restart the active one. Attempting to do so using Launch Debug displays the following error: There is already a debug configuraton "testConfig" running.

Unfortunately the extension does not solve my issue. I like the approach (defining a command for a specific configuration), but the problem is that it always attempts to start the configuration. In my case, I need it to restart the active one. Attempting to do so using Launch Debug displays the following error: There is already a debug configuraton "testConfig" running.

Good point, I'll look into stopping the matching debug session first before re-starting the same configuration.  It looks doable.

Good point, I'll look into stopping the matching debug session first before re-starting the same configuration.  It looks doable.

Cool. Do keep in mind that stopping and starting is not the same as restarting for many browser based debuggers (e.g. Debugger for Chrome, which is used for Electron, reloads the window on restart, but not on re-attach).

Cool. Do keep in mind that stopping and starting is not the same as restarting for many browser based debuggers (e.g. Debugger for Chrome, which is used for Electron, reloads the window on restart, but not on re-attach).

@weinand Looks like implementing this functionality in an extension would require an API to restart a DebugSession. I could imagine this being an option for debug.startDebugging or a separate function. Any chance of that happening?

@weinand Looks like implementing this functionality in an extension would require an API to restart a DebugSession. I could imagine this being an option for debug.startDebugging or a separate function. Any chance of that happening?

Most built-in commands (like "Restart") have an ID ("workbench.action.debug.restart") and can be executed via the vscode.commands.executeCommand API

Most built-in commands (like "Restart") have an ID ("workbench.action.debug.restart") and can be executed via the vscode.commands.executeCommand API

The command you suggested restarts the currently active debugging instance, which is the reason why this ticket exists. There is no way to restart a different instance via commands or the API.

The command you suggested restarts the currently active debugging instance, which is the reason why this ticket exists. There is no way to restart a different instance via commands or the API.

@isidorn any idea how to address this issue?

@isidorn any idea how to address this issue?

@weinand that sounds like a good approach and it might already work looking at the code here
@DJ4ddi you should be able to execute the workbench.action.debug.restart and pass the following object { sessiondId: YOUR_SESSION_ID }
Try passing it as the first argument, if that does not work, pass it as a second argument.
Let me know how it goes.

@weinand that sounds like a good approach and it might already work looking at the code here
@DJ4ddi you should be able to execute the workbench.action.debug.restart and pass the following object { sessiondId: YOUR_SESSION_ID }
Try passing it as the first argument, if that does not work, pass it as a second argument.
Let me know how it goes.

Great, that worked (second argument is the correct one).
Code sample for my use case:
var sessions = [];
function activate() {
  vscode.debug.onDidStartDebugSession(session => sessions.push(session.id));
  vscode.debug.onDidTerminateDebugSession(session => sessions = sessions.filter(sid => sid !== session.id));
  vscode.commands.registerCommand("workbench.action.debug.restartLatest",
    () => vscode.commands.executeCommand("workbench.action.debug.restart", "", { sessionId: sessions[sessions.length - 1]);
  });
}

@ArturoDent this might interest you as well.

Great, that worked (second argument is the correct one).
Code sample for my use case:
var sessions = [];
function activate() {
  vscode.debug.onDidStartDebugSession(session => sessions.push(session.id));
  vscode.debug.onDidTerminateDebugSession(session => sessions = sessions.filter(sid => sid !== session.id));
  vscode.commands.registerCommand("workbench.action.debug.restartLatest",
    () => vscode.commands.executeCommand("workbench.action.debug.restart", "", { sessionId: sessions[sessions.length - 1]);
  });
}

@ArturoDent this might interest you as well.


Explain

Please if possible explain me this library.. I have by module RC522 width UART...
Sorry for my english!!!

Hi MagoKimbra, you have a simple example in the "examples" folder. If you have any specific question, please, ask here and I will try to help you. :)

I set the MISO(TX) pin on D0 and SDA(RX) on D1 but I got nothing... I have the RFID-RC522, do I need to connect 2 pins to activate UART mode ?
Also I've tested my device work with SPI and it's working.
Can you help me pls ?

Hi jreziga, what model of Arduino are you using? Note that some models of Arduino share the Serial Bus (like Arduino nano) with the USB conector. While you are using the USB connection with your computer the Serial Bus with RC522 doesn't works fine. If you are using an Arduino Mega, please use the Serial1 on pins 19 (RX) and 18 (TX). For some other models see https://www.arduino.cc/en/Reference/Serial.
Please, let me know if this resolve your problem.

I'm using Arduino Uno. My NFC reader is that one : http://www.banggood.com/fr/RC522-Chip-IC-Card-Induction-Module-RFID-Reader-p-81067.html

I could not get it to go, because it is set to work in i2c.
I have solved with this http://www.elechouse.com/elechouse/index.php?main_page=product_info&cPath=90_93&products_id=2258&zenid=lemnu9n477rsbvu951fqrmnf25
https://www.youtube.com/watch?v=-PMeuHj56-E

please help me zoider.. i has try simple example in the "examples" folder.. and i'm using MFRC522 with   ArduinoMega2560 I set the MISO(TX) pin to pins 19 (RX) and SDA(RX) pin to pins 18 (TX).. is this the correct configuration? I haven't got any response when opening the serial monitor..

this works with this module ?
http://www.seekic.com/uploadfile/member_product/5199/20138242131767.jpg

Hi @yoonzy. Your configuration seems correct. But remember to change "&Serial" on the example code to "&Serial1" if you are using pins 19 (RX) and 18 (TX) with Arduino Mega.
Hi @ricardojlrufino. My module is not exactly that one (is this), but you can try it and tell me if it works. :)

I think that my module is for SPI only

Hi #zodier, i using mega2560 and rfid_module_v4 elechouse, i don't know the way to write a byte to card, i can write by SPI comm, can you teach me the way to write by uart, thank !

I'm trying to understand how should I talk to this chip via UART and one evening was not enough for this.
I've found a question. Where does the 0xAB came from? As I see in the datasheet, to read a register you should send zero, then 8 bit is data then one. So one the package is 10bit. And your code looks weird, I can't understand what do you send and why should it work. Also I don't see the serial number request, why do you expect the chip will send it on init?

Hi zodier !
I’m sorry because now I reply, I just don’t understand the way to send & receive module MFRC522 by UART, also if by SPI I can write the number to card very easy
Or read number of card again, but I think speed R/W SPI slower UART, can you teach me the way write/read data to card, example
Write number 1 to card, after that read card.
Now I have project make auto car, Car will full other car to transfer material. The auto car will read card to know position such as warehouse or station.
When I using SPI, sometime auto car pass card, so I think using uart will be better
Help me pls !
From: Himura Kazuto<EMAIL_ADDRESS>Sent: Thursday, January 26, 2017 6:36 PM
To: zodier/MFRC522-UART-Arduino
Cc: nguyentuancdt1; Comment
Subject: Re: [zodier/MFRC522-UART-Arduino] Explain (#1)
I'm trying to understand how should I talk to this chip via UART and one evening was not enough for this.
I've found a question. Where does the 0xAB came from? As I see in the datasheet, to read a register you should send zero, then 8 bit is data then one. So one the package is 10bit. And your code looks weird, I can't understand what do you send and why should it work. Also I don't see the serial number request, why do you expect the chip will send it on init?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/zodier/MFRC522-UART-Arduino/issues/1#issuecomment-275368273 , or mute the thread https://github.com/notifications/unsubscribe-auth/AXpUqj6eqDbvDVl2xMNKVKtJeshedCqEks5rWIUKgaJpZM4HJ4Wv .  https://github.com/notifications/beacon/AXpUqhbWNwa1Qo_5QZLVtuXwo3VeAjLuks5rWIUKgaJpZM4HJ4Wv.gif

Use my repo, bro. I did it, my RC522 is working via UART. I found that the usual code that everybody use is quite weak, but as I googled, there's no alternatives((

Oh yes, I already see your code, now I have a mega2560 board, I’m using TX1/RX1(pin18,18) connect to MRFC522, TX0/RX0 to show serial but I don’t know change in your code to reconfig, I think have to change in  https://github.com/zodier/MFRC522-UART-Arduino/blob/master/MFRC522.cpp MFRC522.cpp and  https://github.com/zodier/MFRC522-UART-Arduino/blob/master/MFRC522.h MFRC522.h also Conector.begin(&Serial); allright ?
If I want to write number 1 into card, is it “Connector.write(1)”  ???
If I want to read and show the number of card on Serial0, is it “Serial.println(Connecto.read)”  ???
Can you teach me the way to change, thank bro !
From: Himura Kazuto<EMAIL_ADDRESS>Sent: Thursday, February 02, 2017 2:58 PM
To: zodier/MFRC522-UART-Arduino
Cc: nguyentuancdt1; Comment
Subject: Re: [zodier/MFRC522-UART-Arduino] Explain (#1)
Use my repo, bro. I did it, my RC522 is working via UART. I found that the usual code that everybody use is quite weak, but as I googled, there's no alternatives((
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/zodier/MFRC522-UART-Arduino/issues/1#issuecomment-276891228 , or mute the thread https://github.com/notifications/unsubscribe-auth/AXpUqpGtCHKJPMqASUuKrbmpDDU7aoX2ks5rYYyWgaJpZM4HJ4Wv .  https://github.com/notifications/beacon/AXpUqoElqIwyVUyPToDuveWQE-BMpncRks5rYYyWgaJpZM4HJ4Wv.gif

Use the official repository https://github.com/miguelbalboa/rfid, this code is much more powerful.
First thing you have to do is rewrite the register-reading and -writing functions.
https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L46
https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L61
https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L79
https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L95
Refer to my python code.
And watch all pin-dependant statements, you should correct them also.
like here https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L199
And don't forger to switch your board to UART mode. http://igor-kochet.livejournal.com/267758.html

Exactly, I already use https://github.com/miguelbalboa/rfid this library to SPI comm, read and write data card normal, now I want to use UART comm but can not :(. I use MRFID-RC522 in elechouse it’s not RFID-RC522, it don’t have switch SPI-UART. Haizzz !!!!
Help me bro ^^
From: Himura Kazuto<EMAIL_ADDRESS>Sent: Friday, February 03, 2017 2:13 PM
To: zodier/MFRC522-UART-Arduino
Cc: nguyentuancdt1; Comment
Subject: Re: [zodier/MFRC522-UART-Arduino] Explain (#1)
Use the official repository https://github.com/miguelbalboa/rfid, this code is much more powerful.
First thing you have to do is rewrite the register-reading and -writing functions.
https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L46
https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L61
https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L79
https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L95
Refer to my python code.
And watch all pin-dependant statements, you should correct them also.
like here https://github.com/miguelbalboa/rfid/blob/master/MFRC522.cpp#L199
And don't forger to switch your board to UART mode. http://igor-kochet.livejournal.com/267758.html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/zodier/MFRC522-UART-Arduino/issues/1#issuecomment-277180044 , or mute the thread https://github.com/notifications/unsubscribe-auth/AXpUqu8D7f4uArpT1XQjUF8LXI4uNdp9ks5rYtONgaJpZM4HJ4Wv .  https://github.com/notifications/beacon/AXpUqpDMTypKBEQbTe6zFvNvavkothcnks5rYtONgaJpZM4HJ4Wv.gif

There's no 'switch'. you will need to cut the pull-up track to pin 32 as shown here http://igor-kochet.livejournal.com/267758.html
Also, I have to notice that the task you are going to achieve is quite complicated. You will need to read the datasheet, understand the existing code (and how it is connected to the datasheet) then write your own code accordng to the datasheet. Of corse now you also have my working code in another language, but it simplifies your task just a little bit. In case something should not work as expected, there will be noone to help except for the datasheet.
You can go on searching for a ready solution in case it's too hard for you.
Or you can drop your arduino and start using the real-world hardware and software. In this case you will have my ready code.

Thank  https://github.com/Himura2la Himura2la !
Does your code have connect between adruino uno and mrfid-rc522 ?
I will review again !

I don't use arduino. My code connects rc522 with a PC or anything that could execute Python. For instance, Raspberry Pi or ESP8266

oh oh, so i don't understand because your code differance adruino, sorry now i'm newbie hihi

That's why I warned you this is not an plug-and-flash task and may require time and knowledge...

ok thank you !
i will trying

@Himura2la can you write us a simple read and write for Arduino using UART? I would be glad even if it is not a complete code.

Sorry, I don't have time to translate my code into the language I hate (
Python is extremely readable, I hope you won't have problems in understanding my code

Okay sorry but maybe I was too demanding. I have never worked with Python but I will give it a shot. Would you mind if I contact you from time to time for a specific question regard with the code?

Sure. You definitely have to try Python, it's just perfect.

i am so confused about  DTRQ and MX pins in UART communication. I cant found it in RC522 module.

They are not required, in fact I never ever used them and even don't know
their purpose.
UART is RX, TX and common GND
On Sun, Mar 11, 2018, 5:58 PM Prasset<EMAIL_ADDRESS>wrote:

i am so confused about DTRQ and MX pins in UART communication. I cant
found it in RC522 module.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zodier/MFRC522-UART-Arduino/issues/1#issuecomment-372121579,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHcPuOIgbzi-A0EvpHpncIkY7FBObFEiks5tdTt8gaJpZM4HJ4Wv
.


Fix the problem with wrong value of isAuthorized()

More details - #69

@dmitriys-lits published to the npm registry! Thanks again! :)


What is the Security guidelines for OLM fixing known CVEs?

Type of question
OLM Security
Question
What is the Security guidelines for OLM fixing known CVEs?
What did you do?
I see the 0.11.0 release out as the latest release, but is it flagged with a CVE:https://quay.io/repository/operator-framework/olm/manifest/sha256:81813ac9c937187c29e080c0975bb18489c1f232009c38c8d3a27bc9956ddd21?tab=vulnerabilities&fixable=true
What did you expect to see?
I expect to see a 0.11.1 release spin to pick up and fix the CVE within a reasonable timeframe (a couple weeks?)
What did you see instead? Under which circumstances?
No new release to pick up the CVE fixes.
Environment

operator-lifecycle-manager version:

Latest 0.11.0

Kubernetes version information:
n/a

Kubernetes cluster kind:
n/a

Additional context

Currently, our upstream releases do not carry an SLA for fixing known CVEs.
This is already fixed in the master tag, and we will be making another minor release within a week or so to fix this.

@ecordell Thx for the info.    Are there plans to put our a SLA statement regarding security/CVEs resolution?     Trying to understand how to consume this for production and have some type of timeframe for getting high severity security fixes.

@ecordell   Friendly reminder that the current 0.12.0 release has 3 CVEs.   Hopefully there will a new release soon.  I checked master just now, and it scanned was clean.

@ecordell   Any updates on new release to pick up CVEs?

@ecordell  Any timeframe for 0.13.0 release?

Working with @dmesser to come up with a process for get release spins.


KP house longitudes are wrong (not giving unequal division , not matching with softwares)

The house api is unable to calculate the placidus (unequal) house division, it is just adding 30deg for each next house

https://[vedastroapi.azurewebsites.net/api/Calculate/HouseDegrees/HouseName/All/Location/Delhi,India/Time/01:30/14/02/2024/+05:30/Ayanamsa/KrishnamurtiKP]

you got the wrong API URL, this means normal astrology not KP, you have to add "KP" to end of the method
did you check with values from this site? --> https://kpastrologyaiwebstorage.z5.web.core.windows.net/Charts.html
please please use the website link above to verify KP rated data before, if you want to verify any API data use that websites edit button that I showed you, switch the API from the drop down menu after clicking the edit button which looks like a pencil

There is no method for finding the start and end longitudes of KP houses (unequal placidus division)

Thank you again for bringing this up.
This issue has been corrected with two new methods one of which you wrote
below is the method names
HouseLongitude & HouseLongitudeKP


[Audio] Refactor paddleaudio to paddlespeech.audio

PR types

Others
PR changes

Others
Describe

Refactor paddleaudio to paddlespeech.audio

TODO: merge audio docs.


Submit on create/edit

When using Firefox 9 and when you try to create/edit something in
the admin panel the submit button doesn't work.
In IE 8 the css breaks completely.
The submit button correspond to a JS call.
BTW I love the way you are going with this new version, any idea of implementing a way
for ordering the admin menu?

Thanks for the error report on this. I'm aware of the submit issue, but haven't tested the layout in IE (running on Mac).
And yes, I'm planning on implementing a way for ordering Menu. It wont just be for the Admin though, it'll be via the Menu module.


[Unity][Disco] separate computation and communication into 2 stream

put computation on default stream and put communication on a new stream.

cc: @junrushao

i'm trying to figure out - which stream is used for compute and which is for communication? are we using null stream for compute?

null stream (or default stream equivalently) is used for compute. The stream created in init_ccl is for communication


[BUG] text with overlapping characters and distorted formatting

Description
The app occasionally displays text incorrectly with overlapping characters and distorted formatting, as seen in the attached image (table from the list of files in a collection). This affects the readability and usability of the interface, particularly when handling PDF documents or lists with long filenames.
Reproduction steps
1. Go to a collection and upload files with long file names (eg 40 chars)
2. See the table with list of files and bad formatting

Screenshots
![Example screenshot](https://github.com/user-attachments/assets/49dc58d6-9738-4563-bb38-dbd2ff37b762)

Logs
No response
Browsers
Firefox
OS
MacOS
Additional information
No response

Safari doesn't exhibit the same issue.

Firefox on Linux has this issue too.

I don't see this text display issue anymore.  Fixed?
I'm running latest Docker main-full image.


Validate tx

Draft PR for first round feedback

Good point. I changed to allow Coinbase and added this to be used in the
Coinbase validation. The Coinbase validation is actually stricter and is
done separately. So I may make this strict and not allow empty inputs
On Wed, Mar 23, 2022, 5:46 PM Eamon Ito-Fisher @.***>
wrote:

@.**** commented on this pull request.
In src/core/txs/validate_tx.c
https://github.com/olincollege/SoftSysOlinCoin/pull/20#discussion_r833749295
:

int input_unlockable = check_input_unlockable(input, blank_tx_hash);
+}

+int validate_tx_not_null(Transaction *tx){

if(tx->num_inputs != 0){
return 1;
}
if(tx->inputs != NULL){
return 2;
}
if(tx->num_outputs != 0){
return 3;
}
if(tx->outputs != NULL){
return 4;
}

Doesn't seem to break anything, seems fine to me. If you wanna update the
serialization to not Malloc when num is 0 that seems alright too.
I'm alright leaving as is and not handle coinbase, but would prefer a
comment pointing that out if so.
—
Reply to this email directly, view it on GitHub
https://github.com/olincollege/SoftSysOlinCoin/pull/20#discussion_r833749295,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFOJTIVKHEK56NV7YZ23DHLVBOGMLANCNFSM5RJTE5FA
.
You are receiving this because you authored the thread.Message ID:
@.***>


AnchorTagHelper: seealso usage

Add an external reference to Anchor Tag Helper in ASP.NET Core to the page AnchorTagHelper Class

Docs Build status updates of commit 2675d8b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnet-core/xml/Microsoft.AspNetCore.Mvc.TagHelpers/AnchorTagHelper.xml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

per the readme:
API PR's ( api/autoapi/... ) should not be made here. Changes to API documentation should be made in the source repository against the triple slash /// comments.


Fix ubuntu18

Some changes for building on ubuntu 18.04.
There was a strange race condition in scons -j3 with the vjmol directory that only appeared on ubuntu18 (oddly not on 14 or 16)  if the vjmol directory didn't exist then the Jar() for vjmol.jar created a dysfunctional build command.

I like the Execute('find ...') change in the biopack SConstruct, especially for the "templates" files where I had to use multiple Execute('chmod ...') calls because the lists were too long. However, all of the files in the bin directory need to have the execute bit turned on. Also, if scons decides to execute the SConstruct for psglib before doing it for biopack, then the Execute('find ...') will also turn off the execute bit for all the files in seqlib, which it should not do. I think the SConstructs for biosolidspack, solidspack, veripulse, etc may have similar issues, especially with respect to seqlib.
I am travelling starting tomorrow and do not have enough time to review all the changes. I will pick it up again when I return on July 10.

The find/chmods are unnecessary, I added them in the process of tracking down the vjmol problem.  I'll take them out for now.


Apply fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Coverage remained the same at 30.169% when pulling 9b0dbbb5897f99bb86bb0ba35b8f82110e3539ce on analysis-qrwVED into 00ea1cbff3caf11ce0199f4babd335418e5995ed on master.


Do we need a cita_play image?

Description
A full image including all the dependent parts of CITA. You can play CITA using this image without other operation. such as TiDB of PinCap.
Now we need to pull the cita_build after we git clone the code if we want to use the latest branch.
Pull the cita_run image to run the CITA if we use the  release version.
Additional Information
-------------------Chinese--------------
我们是否需要一个开箱即用的镜像？目前的情况是如果我们用最新的 develop 分支，就得拉取 cita_build 镜像编译，如果想要使用发布版，需要拉取 cita_run 镜像运行

Yes, we need it!
We need to release that image which has fully tested.

+1, and I prefer cita than cita-run as the docker image name

env.sh and daemon.sh are not friendly for users. It's ok for developer, not users. Users will be confused.

@ouwenkg

Yeap， I had been bothered by env.sh or daemon.sh for a long time. If you start your node use command .env.sh ./bin/cita start test-chain/*,  node processes does not start and no error msg. That so terrible for users or external developer.


auth: add span to FetchToken helpers

Before this, during a call to the docker resolver, we would generate span wrappers for each HTTPRequest correctly, however, as the docker resolver reaches out to the docker authorizer, it could create HTTP requests (for fetching tokens) that would not be wrapped in any span.
This can result in rather confusing traces, e.g. something like:
remotes.docker.resolver.HTTPRequest
	HTTP HEAD (fetch index, fails with 401)
HTTP GET (fetch token)
remotes.docker.resolver.HTTPRequest
	HTTP HEAD (fetch index)
remotes.docker.resolver.HTTPRequest
	HTTP GET (fetch manifest)

By adding a span into the FetchToken, this trace becomes a little easier to consume:
remotes.docker.resolver.HTTPRequest
	HTTP HEAD (fetch index, fails with 401)
remotes.docker.resolver.FetchToken
	HTTP GET (fetch token)
remotes.docker.resolver.HTTPRequest
	HTTP HEAD (fetch index)
remotes.docker.resolver.HTTPRequest
	HTTP GET (fetch manifest)

I think this is the right kind of approach for this. However, maybe it makes more sense to add this in the authHandler instead? Happy to discuss :tada:

Bump - any issues with this?


[Feature Request] Mariner's tdnf repo should have specific versions of Python - python39

Red Hat Enterprise Linux has Python 3.6, 3.8, and 3.9 available in their yum repo: https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/8/html/configuring_basic_system_settings/assembly_installing-and-using-python_configuring-basic-system-settings
The user can choose which version of Python to install using python3, python38 and python39.
Mariner, however, only has one python3 which is 3.7 on Mariner 1.0 and 3.9 on Mariner 2.0.
# tdnf install python39
Loaded plugin: tdnfrepogpgcheck
python39 package not found or not installed
Error(1011) : No matching packages

It would be great if Mariner can also release python39 package. This can make our build process easier (https://github.com/Azure/azure-cli/pull/22606).

Explicit is better than implicit.
-- PEP 20 – The Zen of Python

Also see:

https://github.com/microsoft/CBL-Mariner/issues/2860

Thanks for the suggestion. I'll add the proper Provides statement in the spec file during the next update to the python package.

Feature implemented, thanks for the feedback!


Fix GH actions to run for v2.1 too

What this PR does:
Fixes Github action applicability to get API tests to run for v2.1 too, not just v1/main/PRs.
Which issue(s) this PR fixes:
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

Problems with DSE backend remain, but not caused by this PR; merging since API tests are needed by other PRs.

These changes must be on main as well.. But it might work for pull requests if the base is v2.1.. Should be checked..

This has always been bit confusing for me, but I think it's enough for changes to be in the branch being pushed (wrt pushes); for PRs applicability was already not limited.
But if it turns out it's not enough I (or whoever has chance) can merge it to main for sure.

When merging from main to v2.1, Coordinator test CI run is now triggered so I think we are good.


Add null checks for org.springframework.boot.autoconfigure.web.ServerProperties.Servlet#getContextPath()

Fixes gh-3259

Codecov Report

Merging #3262 into master will increase coverage by 0.49%.
The diff coverage is 88.88%.

@@             Coverage Diff              @@
##             master    #3262      +/-   ##
============================================
+ Coverage     65.46%   65.95%   +0.49%
- Complexity     1475     1478       +3
============================================
  Files           188      188
  Lines          6943     6949       +6
  Branches        846      849       +3
============================================
+ Hits           4545     4583      +38
+ Misses         2085     2052      -33
- Partials        313      314       +1

Impacted Files
Coverage Δ
Complexity Δ

...cloud/netflix/zuul/ZuulProxyAutoConfiguration.java
80.76% <100%> (+3.49%)
9 <4> (+2)
:arrow_up:

...loud/netflix/zuul/ZuulServerAutoConfiguration.java
75.51% <66.66%> (-1.09%)
16 <1> (ø)

...cloud/netflix/eureka/EurekaInstanceConfigBean.java
69.03% <0%> (+16.75%)
54% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6859b7a...d815bb5. Read the comment docs.

Ping @candrews, could you add a test?


Support Previews for Edit Based Code Actions

Summary of the changes

LSP Platform doesn't currently support previews for WorkspaceEdit.Changes. Transitioned to WorkspaceEdit.DocumentChanges to ensure we get code action previews.

Fixes: https://github.com/dotnet/aspnetcore/issues/25894

:shipit: assuming all still works in VSCode

Yep!


move frame ordering to producer. Apply timecode scale to frames pts b…

…efore putFrame to avoid fragment overlap issue.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Can we tag the github issue in the description?


How can we change video videoId state and play it at the same time?

I am following the example of this repo.
Not working code attempt 1:
  onChangeVideo() {
    this.setState({
      videoId: this.state.videoId === videoIdA ? videoIdB : videoIdA,
    });
    this.onPlayVideo(); //this do nothing
  }

Not working code attempt 2:
  onChangeVideo() {
    this.setState({
      videoId: this.state.videoId === videoIdA ? videoIdB : videoIdA,
    },()=>{
      console.log(this.state.videoId)
      this.onPlayVideo();//still do nothing
    });
  }

Working code,  attempt 3:
  onChangeVideo() {
    this.setState({
      videoId: this.state.videoId === videoIdA ? videoIdB : videoIdA,
    });
    setTimeout(()=>{
          this.onPlayVideo();
    },0)// this hack make it work
  }

On non-mobile browsers you can rely on the autoplay player var to start playing the video: https://github.com/tjallingt/react-tjube/blob/master/src/components/Player/Player.jsx#L13


Rendering large text.

Hi there,
I’m working on a project where I need to render large Markdown documents into HTML using Markdig. The issue I’m running into is performance when re-rendering the entire document after every change. I’d like to optimize this process by only re-rendering the sections that have been modified or added.
I’m wondering if there’s a way to achieve this using or any suggestions?
Cheers.

I’m wondering if there’s a way to achieve this using or any suggestions?

It would require maintaining the previous document model in memory and performing your own diff, and removing the nodes that are the same and then re-render things... but yeah, it will be complicated. You will need to identify where previous blocks map to previous text in order to do a safe replacement. It's possible (e.g. you can hook into the HTML renderer to record the line + associated markdown block)
How large is your input document and how frequent do you have to re-render it?


Fixed Verbose Statement Returning Incorrect Variable

Line 553 was returning $notificationCompliant instead of $sourceCompliant in the verbose statement related to the Source/service check.

This change is

Codecov Report

Merging #83 into dev will not change coverage.
The diff coverage is 88%.

@@        Coverage Diff        @@
##           dev   #83   +/-   ##
=================================
  Coverage   62%   62%
=================================
  Files        2     2
  Lines      307   307
=================================
  Hits       191   191
  Misses     116   116

You can cherrypick just the edits to line s 447 and 553 if that helps, the rest of the changes are automatic formatting crap that VSCode did on it's own.

Not abandoned. Can someone please take a look here?

Sorry @Chirishman I've disabled the stale bot for now, and will work through this soon...
Removing the Stale label for now.
A quick review seems good but all the cosmetic changes (which are nice anyway) makes it harder to review. Will get back to it once rested.


Improve LC Pokemon Random Movepools

Updating, creating from scratch, or otherwise modifying most of the LC Pokemon random movepools to make them better-fitted/better for a potential future LC Random Battle, should it occur.

Get Quote to review.

Are we still doing this?

it's less pressing given that lc's already in BF, but it definitely couldn't hurt to have actual random movepools in case we want LC Random Battle as a mainstay or seasonal
i remember quote saying he skimmed through it and it looked fine, and remember someone telling me lc people were going through it but haven't heard back. in any event if we do decide to push through with this format later we can get more thorough review then or as we see the sets pan out

We can revisit this next gen if there's interest in a Random LC format.


How to implement keyboard shortcut for undo and redo

Look up on the internet how to implement keyboard shortcuts in web pages
Read the docs to find the undo and redo commands in the API


[macOS] Update activation hack to work on Ventura.

Seems like it's now having issues with activation even if app is bundled, but started using executable name instead of bundle name.
Fixes https://github.com/godotengine/godot/issues/68745

Thanks!

Cherry-picked for 3.6.

Cherry-picked for 3.5.2


Patch to compute the volume and area of convex hulls

As mentioned in issue #5013 , I would like to be able to contribute a small patch enabling the volume and area computation of convex hulls.
As this is already implemented in qhull, I added the correct bindings and a simple unit test. Unfortunately, I don't have Cython 0.22 on my Ubuntu machine, meaning that I could not build against the most recent version of scipy but test passed on older versions and I don't think qhull has changed much since.
Please tell me if I can improve this in any way.
Hervé Audren

Looks good to me. As a general rule, I would use assert_allclose(a, b, rtol=1e-14) for comparing floating point numbers, instead of
assert_equal(a, b).

Ok, merged in a254ee1051e8b07 with minor additions.

Ok thank you ! I also just added another test from random input on my branch (This was to compare it with my old and bad bindings of libqhullcpp), feel free to add it if you think it is useful.

That could be useful --- you can send it in a second pull request.

Thanks for this patch, I was sorely missing this method!


[Linux] User is able to Edit Trust in Bank >> Accounts, Validators, Banks,  section

Bug Description
User is able to Edit Trust in Bank >> Accounts section
Steps to Reproduce

Go to Banks >> Accounts
Check "Trust" column

Expected behavior
User should not be  able to Edit Trust in Bank >> Accounts section
Actual behavior
User is able to Edit Trust in Bank >> Accounts section
Screenshots/Recordings

OS and Browser

Linux  5.0

Additional context
App Version : 1.0.0-alpha.22
Account Number
15e164325715c9552911caf7e3ea1759f4c2ccba5fd82864a4d811d67935c4f9

When you are authenticated then you are able to edit trust.


Added .editorconfig file for use with EditorConfig plugin

This will ensure that editing retains the current indentation etc when used with the EditorConfig plugin or an editor that natively supports it.

Lovely stuff


Multilingual Bot - Greeting Trigger - Conversation Update Activity showing english, not the language selected

Describe the bug
Multilingual Bot - Greeting Trigger - Conversation Update Activity showing english, not the language selected
I am using en-US and hi-IN, when I send hi-IN from webchat - directline, it shows greeting trigger/ actions in english and rest all in triggers in hindi

Version
Version:  2.1.1-nightly.267886.6a6a24a
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0

Browser

[ ] Chrome
[ ] Edge

OS

[ ] Windows

To Reproduce
Steps to reproduce the behavior:

Go to Bot
Create two triggers
Trigger 1 with Greeting in English and Hindi files (both lg and lu files)
Trigger 2intent recognized trigger  English and Hindi files (both lg and lu files)
window.WebChat.renderWebChat(
{
directLine: window.WebChat.createDirectLine({
token: 'xxxxxx'
}),
userID: 'WebChat_UserId',
locale: 'hi-IN',
username: 'Web Chat User',
// locale: 'en-US',
// Passing 'styleSet' when rendering Web Chat
styleSet
},
document.getElementById('webchat')

Expected behavior
Greeting should show in english if locale is english and else hindi
But it always shows english irrespective of the locale selected
Screenshots

Additional context
It happens only with Directline webchat , if you test through emulator it works fine

Thanks for reporting this issue @aparnaji .
This is a known issue with Web Chat. Emulator manages the locale so that's why the expected behavior is observed when using Emulator, however the Direct Line channel does not know the locale at the time of initializing the conversation and sending the conversation update to the bot.
As a workaround, you can try implementing a custom welcome message event as demonstrated in this sample:
https://github.com/microsoft/BotFramework-WebChat/tree/main/samples/04.api/a.welcome-event

@stevkan can you please keep an eye on this?

Thanks for update, Issue that I have is only the greeting dialog has this issue subsequent dialogues are coming in right locale

@aparnaji, test if this option works for you. In my own testing, it works. FYI, this assumes you have implemented the custom welcome message event referenced above in the 'a.welcome.event' sample.

In the Greeting trigger (ConversationUpdate activity), remove or disable all actions within it
Add an Event received trigger (Event activity)
In the event trigger:
a. Click add an action
b. Select Create a condition
c. Choose 'Branch: If/else'
d.  In the Condition field, enter turn.activity.name == "webchat/join"
Under True of the 'Branch: If/else':
a. Click add an action
b. Select Create a condition
c. Choose 'Branch: Switch (multiple options)'
d. In the Condition field, enter turn.activity.locale
Under Cases, add a Value for 'hi-IN' and 'en-US'
For each case, add the action(s) you want to have happen there (i.e. the type of greetings you want displayed)

At this point, when Web Chat establishes a connection to Direct Line, the bot will still receive the conversationUpdate. Since the code contained within that trigger was disabled/removed, the bot will essentially skip it.
Once the connection to Direct Line has completed, Web Chat will fire off an event activity with the name value of webchat/join and a value of window.navigator.language. Note: the 'a.welcome-event' sample sends an event with a payload value of { window.navigator.language }. I chose NOT to send it as an object and, instead, just sent the string.
The bot, looking for a specific event that has the locale defined in the value property, will filter on the locale displaying the appropriate greeting message, as a result.

Closing as resolved.


can't instantiate NetObserv eBPF Agent on kernel 5.4

I'm trying to run it on kernel 5.4 (Ubuntu 20, 5.4.0-189-generic) and it fails with
FATA[0000] can't instantiate NetObserv eBPF Agent        error="loading and assigning BPF objects: field EgressFlowParse: program egress_flow_parse: map direct_flows: map create without BTF: invalid argument"
Is there any way to make it work because upgrading all the servers is not realistic atm.

Well, it turns out is not that hard. I had to replace the RingBuffer with PerfEvent Array in the ebpf code, remove the fentry which is not supported yet and do few changes to use the perf reader.
If anyone is interested I can share more details.

Thanks @matijavizintin for reporting the issue. We indeed test & validate the agent on more recent kernel versions, sorry for the quirks you had with an older one.
cc @msherif1234
I'm actually wondering why we claim supporting kernels 4.18+ (this is in the readme) as the ringbuffer was only added in 5.8, and iirc the very first version of the agent was already using the ringbuffer. Maybe some historical knowledge that we lose...
We do some specific reassignments for kernels older than 5.14 but that's not related to the ring buffer.
@matijavizintin , would your changes be straightforward to integrate upstream?
On the fEntry, a quick look suggests that we fall back to using kprobe when they fail, is there something missing here?

It was a fun one and I learned something new :)
I would say yes. I did it in a hackish way because I needed to prove that it works but in general I added a perf reader to FlowFetcher to read from BPF_MAP_TYPE_PERF_EVENT_ARRAY instead of BPF_MAP_TYPE_RINGBUF.
Regarding fEntry that's true however the code will already fail when loading TCPRcvFentry ebpf program. So I commented that out and used kprobe.
I'm attaching the patch so you can see what I did. As I said, very hackish.
support_for_older_kernels.patch

we already fall back to kprobe if fentry isn't supported or available in fact I remember fentry not available for s390 arch even with recent kernel see https://github.com/netobserv/netobserv-ebpf-agent/pull/265
readme probably need some updates I think we need to set mini kernel version to 5.8
we can't switch our ringbug map with perf events in production as it less efficient for our application
https://nakryiko.com/posts/bpf-ringbuf/

its a warning not an error https://github.com/netobserv/netobserv-ebpf-agent/blob/main/pkg/ebpf/tracer.go#L175 and we do use this logic in production, for older kernel check this is a bug IMO and https://github.com/netobserv/netobserv-ebpf-agent/pull/374 should help will you able to see if that helps with your kernel ?

@msherif1234 if there is a demand (upstream) to keep support for 5.4 / 5.8, etc. , we could create one or more dedicated branches. Which also allows us to clean up the main branch and get rid of the compatibility code, wdyt?

@jotak if want to go to older kernel w/o rbuf support we might have to do a fair bit of work to replace with perf event map type which will require good amount of work in ebpf and the userspace ? but its possible if there is pressing need

@msherif1234 sorry for the late reply. I applied your changes from https://github.com/netobserv/netobserv-ebpf-agent/pull/374 to my branch and it works well, thanks!

Thank you @matijavizintin !!


Packets for ExternalIP are not handled by cilium

Is there an existing issue for this?

[x] I have searched the existing issues

What happened?
My deployment is based on K3S with a single node environment. This node bundles a openvpn connection to a MGMT infrastructure, and two redundant tunnel are closed (tun0, tun1); for routing purpose a "vip" ip is assigned to the lo iface.
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/32 brd <IP_ADDRESS> scope global lo:10
       valid_lft forever preferred_lft forever
7: tun0: <POINTOPOINT,MULTICAST,NOARP,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UNKNOWN group default qlen 100
    link/none
    inet <IP_ADDRESS>/22 brd <IP_ADDRESS> scope global tun0
       valid_lft forever preferred_lft forever
8: tun1: <POINTOPOINT,MULTICAST,NOARP,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UNKNOWN group default qlen 100
    link/none
    inet <IP_ADDRESS>/22 brd <IP_ADDRESS> scope global tun1
       valid_lft forever preferred_lft forever

Assigning this ip as the node's externalIP and then using MetalLB to expose different services based on the listening ifase result in RESET connection.
NAME          TYPE           CLUSTER-IP      EXTERNAL-IP       PORT(S)                                     AGE
traefik-lan   LoadBalancer   <IP_ADDRESS>    <IP_ADDRESS>     80:56171/TCP,443:32611/TCP                  3d2h
traefik-vpn   LoadBalancer   <IP_ADDRESS>   <IP_ADDRESS>   4443:54431/TCP,80:58088/TCP,443:36351/TCP   3d2h

18:36:37.172570 IP NAT-IP.53648 > <IP_ADDRESS>.443: Flags [S], seq 290381044, win 65535, options [mss 1250,nop,wscale 6,nop,nop,TS val<PHONE_NUMBER> ecr 0,sackOK,eol], length 0
18:36:37.172590 IP <IP_ADDRESS>.443 > NAT-IP.53648: Flags [R.], seq 0, ack 290381045, win 0, length 0

Instead, reaching via the internalIP I can reach my traefik-lan pod.
Cilium Version
Client: 1.12.3 1c466d2 2022-10-12T11:33:37+01:00 go version go1.18.6 linux/amd64
Daemon: 1.12.3 1c466d2 2022-10-12T11:33:37+01:00 go version go1.18.6 linux/amd64
Kernel Version
5.4.217-1.el7.elrepo.x86_64
Kubernetes Version
Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.5+k3s1", GitCommit:"405bf79da97831749733ad99842da638c8ee4802", GitTreeState:"clean", BuildDate:"2021-12-18T00:43:30Z", GoVersion:"go1.16.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.5+k3s1", GitCommit:"405bf79da97831749733ad99842da638c8ee4802", GitTreeState:"clean", BuildDate:"2021-12-18T00:43:30Z", GoVersion:"go1.16.10", Compiler:"gc", Platform:"linux/amd64"}
Sysdump
No response
Relevant log output
No response
Anything else?
Cilium config
apiVersion: v1
data:
  agent-not-ready-taint-key: node.cilium.io/agent-not-ready
  arping-refresh-period: 30s
  auto-direct-node-routes: "false"
  bpf-lb-external-clusterip: "false"
  bpf-lb-map-max: "65536"
  bpf-lb-sock: "false"
  bpf-map-dynamic-size-ratio: "0.0025"
  bpf-policy-map-max: "16384"
  bpf-root: /sys/fs/bpf
  cgroup-root: /run/cilium/cgroupv2
  cilium-endpoint-gc-interval: 5m0s
  cluster-id: "0"
  cluster-name: default
  cluster-pool-ipv4-cidr: <IP_ADDRESS>/16
  cluster-pool-ipv4-mask-size: "24"
  custom-cni-conf: "false"
  debug: "false"
  devices: eth0 eth1 tun+ lo
  disable-cnp-status-updates: "true"
  disable-endpoint-crd: "false"
  enable-auto-protect-node-port-range: "true"
  enable-bgp-control-plane: "false"
  enable-bpf-clock-probe: "true"
  enable-endpoint-health-checking: "true"
  enable-health-check-nodeport: "true"
  enable-health-checking: "true"
  enable-hubble: "true"
  enable-ipv4: "true"
  enable-ipv4-masquerade: "true"
  enable-ipv6: "false"
  enable-ipv6-masquerade: "true"
  enable-k8s-terminating-endpoint: "true"
  enable-l2-neigh-discovery: "true"
  enable-l7-proxy: "true"
  enable-local-redirect-policy: "false"
  enable-policy: default
  enable-remote-node-identity: "true"
  enable-svc-source-range-check: "true"
  enable-vtep: "false"
  enable-well-known-identities: "false"
  enable-xt-socket-fallback: "true"
  hubble-disable-tls: "false"
  hubble-listen-address: '[::]:4244'
  hubble-metrics: dns drop tcp flow port-distribution icmp http
  hubble-metrics-server: :9965
  hubble-socket-path: /var/run/cilium/hubble.sock
  hubble-tls-cert-file: /var/lib/cilium/tls/hubble/server.crt
  hubble-tls-client-ca-files: /var/lib/cilium/tls/hubble/client-ca.crt
  hubble-tls-key-file: /var/lib/cilium/tls/hubble/server.key
  identity-allocation-mode: crd
  install-iptables-rules: "true"
  install-no-conntrack-iptables-rules: "false"
  ipam: cluster-pool
  kube-proxy-replacement: strict
  kube-proxy-replacement-healthz-bind-address: '[::]:10256'
  monitor-aggregation: medium
  monitor-aggregation-flags: all
  monitor-aggregation-interval: 5s
  mtu: "1500"
  node-port-bind-protection: "true"
  nodes-gc-interval: 5m0s
  operator-api-serve-addr: <IP_ADDRESS>:9234
  preallocate-bpf-maps: "false"
  procfs: /host/proc
  remove-cilium-node-taints: "true"
  set-cilium-is-up-condition: "true"
  sidecar-istio-proxy-image: cilium/istio_proxy
  synchronize-k8s-nodes: "true"
  tofqdns-dns-reject-response-code: refused
  tofqdns-enable-dns-compression: "true"
  tofqdns-endpoint-max-ip-per-hostname: "50"
  tofqdns-idle-connection-grace-period: 0s
  tofqdns-max-deferred-connection-deletes: "10000"
  tofqdns-min-ttl: "3600"
  tofqdns-proxy-response-max-delay: 100ms
  tunnel: vxlan
  unmanaged-pod-watcher-interval: "15"
  vtep-cidr: ""
  vtep-endpoint: ""
  vtep-mac: ""
  vtep-mask: ""
kind: ConfigMap
metadata:
  name: cilium-config
  namespace: cilium-system

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I think the issue are different, I'm using "strict" kubeproxy replacement and as seen in tcpdump the packets are directly dropped.
KubeProxyReplacement Details:
  Status:                 Strict
  Socket LB:              Enabled
  Socket LB Protocols:    TCP, UDP
  Devices:                eth0 <IP_ADDRESS>, lo <IP_ADDRESS>, tun0 <IP_ADDRESS>, tun1 <IP_ADDRESS>
  Mode:                   SNAT
  Backend Selection:      Random
  Session Affinity:       Enabled
  Graceful Termination:   Enabled
  NAT46/64 Support:       Disabled
  XDP Acceleration:       Disabled
  Services:
  - ClusterIP:      Enabled
  - NodePort:       Enabled (Range: 30000-32767)
  - LoadBalancer:   Enabled
  - externalIPs:    Enabled
  - HostPort:       Enabled


Why did you guys have to change the checkbox

I used to like this thing about checkboxes, where using [ ], [-] and [/] gave different visual outputs. And that was really awesome. Imagine my surprise when I bought a new PC and installing this theme doesn't differentiate them anymore. Sucks. Like, why would you remove one of the features that gave you an edge over other themes?

After updating it, it worked. Thanks.
On Tue, Nov 12, 2024, 17:04 Michael Bao @.***> wrote:

It wasn't removed; I just had it disabled by default. It should be enabled
by default with the newest version, so reinstalling the theme should fix it.
—
Reply to this email directly, view it on GitHub
https://github.com/tcmmichaelb139/obsidian-tokyonight/issues/9#issuecomment-2470620209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALMGUEIFC4H3GJPFQBKAWBL2AIDGJAVCNFSM6AAAAABRTL7ZL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZQGYZDAMRQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Make remote data local

Commander Skill
Achievements
Japanese ship names

The Wargaming API is no longer reliable. These values are not changed so often so it is fine to load them locally.

Achievements seems to be fixed for now
Commander Skill should be abandoned because it is too complicated now
Japanese ship names can be downloaded manually later

WoWs-Game-Data will provide all data for upcoming updates.


Auto optimize deps in development

Problem
Workerd cannot run CJS. Certain dependencies are only published in CJS, or are published as ESM but incorrectly call CJS APIs and break at runtime. So far, we need to tell Vite to optimize these dependencies manually by adding them to Vite's ssr.optimizeDeps.include. This is really bad DX and there's not much we can do to figure out what dependencies are like this, because we don't know them until they break the app.
Proposal
This PR catches the errors related to loading files in workerd and tries to figure out what dependencies are related to the error. Then, it adds them automatically to the user's vite.config.js and restarts the server.
Example error in the terminal and browser that we get so far:
ReferenceError: module is not defined
    at /code/node_modules/.pnpm/use-sync-external-store@1.2.0_react@18.3.1/node_modules/use-sync-external-store/shim/with-selector.js:6:3
    at Object.runViteModule (code/node_modules/.pnpm/@shopify+mini-oxygen@3.0.2_vite@5.2.11/node_modules/@shopify/mini-oxygen/dist/vite/worker-entry.js:1181:17)
    at ViteRuntime.directRequest (code/node_modules/.pnpm/@shopify+mini-oxygen@3.0.2_vite@5.2.11/node_modules/@shopify/mini-oxygen/dist/vite/worker-entry.js:1026:78)
    at ViteRuntime.cachedRequest (code/node_modules/.pnpm/@shopify+mini-oxygen@3.0.2_vite@5.2.11/node_modules/@shopify/mini-oxygen/dist/vite/worker-entry.js:949:28)
    at request (code/node_modules/.pnpm/@shopify+mini-oxygen@3.0.2_vite@5.2.11/node_modules/@shopify/mini-oxygen/dist/vite/worker-entry.js:976:128)
    at /@fs/code/node_modules/.pnpm/zustand@4.5.2_@types+react@18.3.1_immer@10.1.1_react@18.3.1/node_modules/zustand/esm/index.mjs?v=2603acb1:3:31
    at Object.runViteModule (code/node_modules/.pnpm/@shopify+mini-oxygen@3.0.2_vite@5.2.11/node_modules/@shopify/mini-oxygen/dist/vite/worker-entry.js:1181:11)
    at ViteRuntime.directRequest (code/node_modules/.pnpm/@shopify+mini-oxygen@3.0.2_vite@5.2.11/node_modules/@shopify/mini-oxygen/dist/vite/worker-entry.js:1026:60)
    at ViteRuntime.cachedRequest (code/node_modules/.pnpm/@shopify+mini-oxygen@3.0.2_vite@5.2.11/node_modules/@shopify/mini-oxygen/dist/vite/worker-entry.js:950:79)
    at /app/lib/app-state.tsx:3:31

This will be now caught, processed, and fixed automatically:

Tophat
Try adding CJS dependencies such as use-sync-external-store (import from use-sync-external-store or use-sync-external-store/shim).
You can also disable any already optimized dep in the Hydrogen plugin (except react/jsx-dev-runtime which is a bit special and doesn't work) to see how they are added to the user config automatically.
This should work ideally without opening the browser. However, if you do open the browser or change imports during development, it should still ideally work.
Note
~I still need to test with PNPM. It might not work there yet because paths in node_modules are a bit different so it might need some extra tinkering.~ I've tested with demo-store + PNPM and it seems to work (at least after this change, but it's probably a rare thing with a very broken dependency).
Errors
Browser error page
Using minimal CSS because this page is not rendered by Remix (plain strings):

When no ssr.optimizeDeps.include array is found in vite.config.ts
The following error shows up. We could try adding it automatically but it might be error prone.

When running with --disable-deps-optimizer flag
We don't write automatically anymore but we still read the error, parse the stack and try to provide helpful information:

Other non-CJS reference errors
Not every ReferenceError comes from an optimizable dependency. If the user has bad code it will be shown like this:

When we can't find the issue
In this situation we tried to optimize textr but that did not solve the issue. In this case, we print the stack for manual fixing. The user should find typographic-base/index in this case:

It looks like warning broke when I just imported it bare. Not sure why this would make a difference


add cloud in environment sky

It would be great if there is paramter-tunable procedural cloud in the default sky, which should be more realistic for the environment

Procedural clouds are highly game-specific, depending on the kind of game you're making you'll likely want completely different types/shapes and styles of clouds, if any. I don't think it should be added to the default sky which is a simple Skybox and should stay so. Maybe it could be added to ProceduralSky, if not available already.

Yes, please add to ProceduralSky. What I mean is ProceduralSky. Sorry for misleading.
And I think with adjustable parameters to adjust the cloud type, size, color, shape, etc., it would be useful for many types of games.
Thank you.

This may have been implemented by #38791, but I'm not sure if the blending scheme works well with cloud panoramas. Either way, it could be achieved by customizing the sky shader.

This discussion is being tracked in https://github.com/godotengine/godot-proposals/issues/835 now


Adds support for configuration files in the scheduler

this is how the config file looks like:
[scheduler]
address = "<IP_ADDRESS>:9000"

[[tasks_settings]]
exec_time = 60
memory = 2
devices = [0, 1, 2]
task_type = "MerkleProof"

[[tasks_settings]]
exec_time = 60
memory = 2
devices = [0, 1, 2]
task_type = "WindowPost"

[[tasks_settings]]
exec_time = 60
memory = 2
devices = [0, 1, 2]
task_type = "WinningPost"

Looks nice.
I wonder if we can use some structured name for devices instead of just a number.
Example
devices = [ /nvidia/gpu/0, /nvidia/gpu/1 ]
something in the form
/{brand}/{type}/{index}

devices = [ /nvidia/gpu/0, /nvidia/gpu/1 ]

I discussed this with @leongb the other day. We agree that using raw numbers is not as user-friendly as we want. but so far, We do not have a tool for mapping devices' names (which are collision prone) to a unique id. or as you propose using /dev/gpu0, /dev/gpu1, or similar.

devices = [ /nvidia/gpu/0, /nvidia/gpu/1 ]

I discussed this with @leongb the other day. We agree that using raw numbers is not as user-friendly as we want. but so far, We do not have a tool for mapping devices' names (which are collision prone) to a unique id. or as you propose using /dev/gpu0, /dev/gpu1, or similar.

Wouldn't it be possible to make a serializer that takes (string?) "/nvidia/gpu/0" and maps that to a unique device ID? i assume the "0" is the unique thing in the string as well

I suppose Natanael means that we can do this in another PR.
But yes, using a serilizer is what I meant. And also the other two elements
would be important so we can differentiate between device types and brands
(example.. if we have two boards amd+nvidia.. which one is 0 and 1?)
We definitely need this URL structure in M2
On Mon, 12 Apr 2021 at 08:49, Leon Groot Bruinderink <
@.***> wrote:

devices = [ /nvidia/gpu/0, /nvidia/gpu/1 ]
I discussed this with @leongb https://github.com/leongb the other day.
We agree that using raw numbers is not as user-friendly as we want. but so
far, We do not have a tool for mapping devices' names (which are collision
prone) to a unique id. or as you propose using /dev/gpu0, /dev/gpu1, or
similar.
Wouldn't it be possible to make a serializer that takes (string?)
"/nvidia/gpu/0" and maps that to a unique device ID? i assume the "0" is
the unique thing in the string as well
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Zondax/filecoin-scheduling/pull/73#issuecomment-817535223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADIQLWABS6PCELS5HCWKI7TTIKJZBANCNFSM42V37E6A
.


Manage custom JWT_PAYLOAD_GET_USER_ID_HANDLER when provided

With reference to the discussion in https://github.com/GetBlimp/django-rest-framework-jwt/issues/196, this pull request uses the custom JWT_PAYLOAD_GET_USER_ID_HANDLER when provided and when custom JWT_RESPONSE_PAYLOAD_HANDLER is missing.

@jpadilla what do you think about this fix? Can we merge this?

@blaklites I don't think this is quite right yet. We'd need to handle both cases in the authentication and serializer. If using JWT_PAYLOAD_GET_USER_ID_HANDLER do left, if using JWT_PAYLOAD_GET_USERNAME_HANDLER do right in this diff. Makes sense?


Run error after git pull

The error I get is:
template: user/dashboard/feeds:42:47: executing "user/dashboard/feeds" at <$push.AvatarLink>: AvatarLink is not a field of struct type *base.PushCommits

Nevermind, I simply deleted my gopath directory and go get -v again. Not sure how I'm supported to keep dependencies up to date along with git pulling gogs.

Just do a go get -u after updating with git and you're done.

:trollface:


Use the updated HubConnection builder pattern in all the clients

In the Java client we introduced a new HubConnection builder pattern where HubConnectionBuilder was an abstract base class for builder implementations. The most common of which being the HttpHubConnection builder.
https://github.com/aspnet/SignalR/blob/c852bdcc332ffb998ec6a5b226e35d5e74d24009/clients/java/signalr/src/main/java/com/microsoft/signalr/HubConnectionBuilder.java#L16-L21
We should use this pattern across the different clients.

There are some issues we need to discuss here, if the factory methods hang off of the builder itself, how do we handle different transports (http vs tcp for example).

For example, in the http vs tcp case we could just use an overload
create(String url)

This is pretty easy to do in TypeScript since the type system is super flexible. We need to think about how it works in .NET, but I think it's achievable.
For example, we can add HubConnectionBuilder.Create() that returns an HttpHubConnectionBuilder but what about the existing extension methods on HubConnectionBuilder, like WithUrl or WithEndpoint. If HttpHubConnectionBuilder inherits from HubConnectionBuilder, this code would work:
var connection = HubConnectionBuilder.Create("http://mysite.com")
	.WithUrl("http://differentsite.com")
	.Build()

That would look weird and be confusing. I think a better approach is probably to have HttpHubConnectionBuilder not inherit from HubConnectionBuilder, but perhaps from some other base class or interface?

@mikaelm12 not quite. Those static methods cannot live on the base builder as they would be in different packages (potentially). I'd like to see a proposal using http and tcp as an example where each of those implementations lived in separate packages.

I'd say we should remove those if we wanted to make HubConnectionBuilder the base class.

No, we can't make a breaking change like that to a core API that's used by everyone. The design needs to be one that doesn't break things. Pick a new base class (HubConnectionBuilderBase) name


Peerflix

Addon Manifest URL
https://peerflix-235ac-default-rtdb.europe-west1.firebasedatabase.app/manifest.json
Addon Description
No response
Language of Content
None
Choose Labels

[ ] anime
[ ] asian drama
[ ] bollywood
[ ] cartoons
[ ] debrid support
[ ] http streams
[ ] live tv
[ ] metadata
[ ] misc
[ ] movies
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[ ] torrents
[ ] tv shows
[ ] video games

this is a duplicate


Crash when scrolling the feed

Crash when scrolling the feed

Can you try to reproduce it with following build:
https://drive.proton.me/urls/VAZWRH29MG#1kvPKw92Maqw
It should be fixed in this build.

I don't know how to reproduce these crashs. So I will close these issues.


[Bug] [1.11.2] [Mixinbooter 4.1] Crash/ Softlock when other player is performing the eating animation

Steps to reproduce:

Enable eating animation
Join multiplayer
Witness someone perform the animation
Crash/softlock should occur

Modlist:
.mixinbooter-4.1
AquaAcrobatics-1.11.2
forge <IP_ADDRESS>60

This will be fixed in 1.11.3.


Adding American Museum of Natural History - Richard Gilder Graduate School

https://www.amnh.edu
https://www.amnh.org/research/richard-gilder-graduate-school
https://www.amnh.org/research/richard-gilder-graduate-school/academics/degrees/phd-comparative-biology
https://www.amnh.org/research/computational-sciences

@nebriv Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


training custom dataset

I have finished writing a brief Chinese blog and introduced the problems I faced in installing and training. I would like to share it so that before the install.md and other guidelines are released, other Chinese students can view it as a substitue.
link: https://blog.csdn.net/JackHu12345/article/details/119881093
Hope I can offer some help!
As for why I didn't write an English version, well , my English is poor.
I'll update my blog if more question is arosen when I'm available.

Thanks a lot for your sincere effort~~

现在有非常棒的install指导啦，俺就先把这个issue关了不献丑啦


Add customizing otlp logs transform to make dynamic loki labels and a…

This allows customizing loki resource labels to add custom filters.
Related issue: https://github.com/grafana/k8s-monitoring-helm/issues/395

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

hi @petewall please can you review?

I finally got caught up on previous PRs, including one that was holding up the rest of them. I'll review as soon as I can.